8000 Test suite · Issue #68 · notable/notable · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Test suite #68

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ghost opened this issue Dec 28, 2018 · 3 comments
Open

Test suite #68

ghost opened this issue Dec 28, 2018 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@ghost
Copy link
ghost commented Dec 28, 2018

I noticed that there are no automated tests for this app. I do test automation for my work, and would be willing to talk about having a test framework set up, if you'd like.

@fabiospampinato
Copy link
Member
fabiospampinato commented Dec 28, 2018

Tests are necessary as the project matures. I think a good first step would be to test the APIs defined in the containers, this alone should help avoid making any regressions.

If anyone would like to take a shot at this we can define some details, I'd usually use AVA for this.

@fabiospampinato fabiospampinato added enhancement New feature or request help wanted Some help is needed and/or PRs are welcome labels Dec 28, 2018
@fabiospampinato fabiospampinato changed the title [QA] Automated tests Test suite Dec 30, 2018
@VictorXLR
Copy link

Hi, could you improve the guide for building this on a window machine.

I tried to build the program on my machine but i kept getting errors when i tried to run

npm run svelto:dev

> svelto build --env development
The filename, directory name, or volume label syntax is incorrect.

I tried to do it in WSL but it had issues rendering due to WSL being linux and i using a windows machine.

Thank you

@fabiospampinato
Copy link
Member
fabiospampinato commented Jan 3, 2019

@VictorXLR that issue is tracked here #41, and it should be already fixed.

@fabiospampinato fabiospampinato added this to the Future milestone Feb 22, 2019
@fabiospampinato fabiospampinato removed the help wanted Some help is needed and/or PRs are welcome label Feb 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants
0