Skip to content
This repository has been archived by the owner on Sep 3, 2020. It is now read-only.

Feedback about document #122

Closed
azu opened this issue Jan 20, 2015 · 3 comments
Closed

Feedback about document #122

azu opened this issue Jan 20, 2015 · 3 comments

Comments

@azu
Copy link
Contributor

azu commented Jan 20, 2015

Hi! Thanks for nice tools ✨
I started to use testium, then I had a couple suggestion.

(My English is not good but hope you understand me...)

1. minimum setup

To work testium took a little longer than I anticipated.
I suggest that prepare officially minimum setup example. (something like jkrems/testium2-demo)

I'v created azu/testium-seed which is a example of minimum setup.

I have wanted to get a example which is actually work.

2. copy/pastable configuration

The Configuration not seem to be copy/pastable.

At first, I pasted the Configuration to .testiumrc and it didn't work.

I was confused.
2015-01-20_15-32-02
(and error message not have much meaning for me...)

I suggest that Configuration is better to be copy/pastable.

Related PR: #123

3. Don't Ignore Errors

The .testiumrc should be error, but only ignore error.

; .testiumrc
launch : true

The .testiumrc should be parse error and output error like follwonig:

launch : true
        ^ Error: Line 2: Unexpected token :

4. Missing wiki?

For help debugging your tests, check out the wiki page.
-- README.md

https://github.com/groupon/testium/wiki/Debugging-Your-Tests is 301.


Maybe these are not actual issue.

Thanks.

@EndangeredMassa
Copy link
Collaborator

Thanks for the reports!

Items 2 and 4 have been fixed. Items 1 and 3 are definitely worth doing.

In the future, please submit a new issue for each item. That will make it easier to track each individual thing.

@azu
Copy link
Contributor Author

azu commented Jan 23, 2015

@EndangeredMassa Thanks!

In the future, please submit a new issue for each item. That will make it easier to track each individual thing.

Sorry, I'll be careful

@EndangeredMassa
Copy link
Collaborator

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants