Comments (13)
It's not fine to have files in PR no one reads and updates blindly when it's broken. It just bring illusion of being tested
from orbit.
Please only if it does not produce one huge snapshot file. No one reads that during code review
from orbit.
@michalsanger Huge snapshot file is fine if it shows only relevant changes. The issue with the implementation on Balkan was that classnames were generated for the entire project instead of each component. So when you updated classnames you would update all of the classnames, rendering it unreadable.
from orbit.
So youβre saying that snapshot tests have no purpose?
from orbit.
Our aim should be to have functioning documentation via Storybook. How do you want to guarantee that all components still render?
from orbit.
I'm saying having huge snapshot file is a problem, because if developer accidentally updates it no one will review those changes, because the file will be big and messy. It's the same with PR with 10 changed files vs 100 changed files. Or how many times do you review content of yarn.lock
?
Having snapshots per component looks better to me.
from orbit.
But Github shows you only the changes between master and the branch. What does it matter if it's big if it shows only the relevant changes?
I can check if Storyshots allows per story files yet.
from orbit.
Are the changes readable? Will you understand that the snapshot change is OK or it's broken? This should come with easy visual verification. Generate storybook for every PR or maybe even faster generate gallery of screenshots
https://www.npmjs.com/package/storybook-chrome-screenshot
from orbit.
Does GitHub allow you to view visual changes between images?
The changes are DOM output changes. They should be readable.
from orbit.
I don't need visual testing π science. I need quick way to see component works. I can spent few seconds running through images when I see something was changed in storyshots. But it would suck if I would have to checkout the branch, install deps, run storybook, find the story...
from orbit.
Does GitHub allow you to view visual changes between images?
Yes, it shows the old and new image (see the commit):
from orbit.
Thanks. We will use the visual storyshot testing.
from orbit.
Closed in favour of #75
from orbit.
Related Issues (20)
- Media query hook not working on initial render. HOT 1
- docs: some Figma iframes require logging in HOT 9
- Prevent drawer overlay close HOT 7
- Padding right added to body when modal opened HOT 4
- Add option for select widget to have icon only
- Possibility to set timeline direction HOT 2
- available `icon list` is broken HOT 2
- ThemeProvider doesn't pass down dictionary prop properly HOT 1
- How to integrate using Rollup with orbit-components as the dependency? HOT 3
- Search shortcut (β + K) doesn't work properly on FF HOT 4
- Accordion Component Broken in Latest Version
- Unable to remove white background on corners of Select component HOT 1
- Popover hijacks space keystroke events HOT 4
- Slider : show the histogram on desktop devices
- "TypeError: Cannot read properties of undefined (reading 'spaceXLarge')" when setting "content" on ItinerarySegmentDetail
- Reporting a vulnerability HOT 1
- align attribute not working in Heading component
- Allow different colored Tile Components HOT 2
- Custom Option filtering for InputSelect HOT 1
- Unexpected error when using OrbitProvider & `useId` prop HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
π Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. πππ
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google β€οΈ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from orbit.