Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add acceptance tests #92

Closed
JDGrimes opened this issue Feb 3, 2016 · 1 comment
Closed

Add acceptance tests #92

JDGrimes opened this issue Feb 3, 2016 · 1 comment
Assignees
@JDGrimes
Labels
enhancement
Milestone
0.1.0

Comments

@JDGrimes
Copy link
Member

JDGrimes commented Feb 3, 2016

We probably can't cover the whole UI at this point, but I'd like to get something started at least. See #91.
@JDGrimes JDGrimes self-assigned this Feb 3, 2016
@JDGrimes JDGrimes added this to the 0.1.0 milestone Feb 3, 2016
@JDGrimes JDGrimes mentioned this issue Feb 6, 2016
Closed
JDGrimes added a commit that referenced this issue Feb 7, 2016
@JDGrimes
Add Codeception acceptance tests
eaee7c8 
I haven’t added the composer config for installing wp-browser and
codeception, because I haven’t figured out how to run the tests on
Travis CI yet, and unless we are running the tests, it is just wasting
time and resources to have composer install the dependencies (and there
are lots of them).

See #92
JDGrimes added a commit that referenced this issue Feb 7, 2016
@JDGrimes
Fix codeception tests output path in .gitignore
9b89648 
See #92
JDGrimes added a commit that referenced this issue Feb 7, 2016
@JDGrimes
Ignore generated codeception files
5709a1d 
See #92
JDGrimes added a commit that referenced this issue Feb 8, 2016
@JDGrimes
Test running Codeception tests with Travis CI
3199d18 
See #92
JDGrimes added a commit that referenced this issue Feb 8, 2016
@JDGrimes
Only run codeception tests on PHP 5.4
7014af7 
Accidentally inverted this check.

See #92
JDGrimes added a commit that referenced this issue Feb 8, 2016
@JDGrimes
Divert phantomjs output
d5c80d0 
See #92
JDGrimes added a commit that referenced this issue Feb 11, 2016
@JDGrimes
Bail wpcept tests on Travis when not PHP 5.4
d14027b 
We forgot to `return`.

See #92
JDGrimes added a commit that referenced this issue Feb 11, 2016
@JDGrimes
Give PhantomJS some time to start
ca36c24 
Otherwise Codeception will think that it isn’t running even though it
is.

And don’t divert its output, so that we can debug it.

See #92
JDGrimes added a commit that referenced this issue Feb 11, 2016
@JDGrimes
Test a fixed version of the php syntax check
de29435 
See #92
JDGrimes added a commit that referenced this issue Feb 15, 2016
@JDGrimes
Use long option to pass mysql password
26493b8 
Because on Travis CI the password is empty, and using the short option
will force the password prompt in that case.

See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Feb 20, 2016
@JDGrimes
Try single quotes for mysqldump password
65adef2 
See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Feb 20, 2016
@JDGrimes
Try omitting the MySQL password when empty
dcfda76 
See #92, #46
JDGrimes added a commit that referenced this issue Feb 20, 2016
@JDGrimes
Don't double quote MySQL password
775e038 
This was failing because the quotes were being added by
`escapeshellarg()`, so we didn’t need to add a second set.

Also, because the value was quoted, the empty value would actually be
equal to `”’’”`, and not `’’`.

Anyway, we leave out the password entirely for `mysqldump`, but leave
it in even if empty for `mysql`, so we can test to see if both ways
work. if they do, we’ll just pass the empty passwords to the commands,
since the logic is simpler.

See #92, #46
@JDGrimes JDGrimes mentioned this issue Feb 26, 2016
Closed
JDGrimes added a commit that referenced this issue Feb 29, 2016
@JDGrimes
Remove debugging code
c8cc1cb 
See #92, #46
JDGrimes added a commit that referenced this issue Feb 29, 2016
@JDGrimes
More debug for the acceptance tests
c0d8e6e 
See #92, #46
JDGrimes added a commit that referenced this issue May 2, 2016
@JDGrimes
WPCept: Print out page source on failure
e4c9da2 
See #92, #46
JDGrimes added a commit that referenced this issue May 5, 2016
@JDGrimes
Pin the WP Browser version
ea05d88 
The latest version is no longer working with the version of Codeception
that we are using (2.1.4). Probably we should update them both, but I
want to get this working again before I mess with that.

The error that currently occurs in Travis:

```
PHP Fatal error:  Class 'Codeception\Application' not found in
/home/travis/build/WordPoints/hooks-api/vendor/lucatume/wp-browser/wpcep
t on line 13

PHP Stack trace:

PHP   1. {main}()
/home/travis/build/WordPoints/hooks-api/vendor/lucatume/wp-browser/wpcep
t:0
```

See #92
JDGrimes added a commit that referenced this issue May 14, 2016
@JDGrimes
Peg WP Browser to a particular release
3df96ee 
The syntax we were using before wasn’t recognized. I guess maybe you
can’t specify a commit hash along with `master@dev`.

See #92
JDGrimes added a commit that referenced this issue May 17, 2016
@JDGrimes
Travis: list codeception output files
c1b4eb1 
The `cat` command below it is complaining that the file doesn’t exist.

See #92
@JDGrimes JDGrimes mentioned this issue May 19, 2016
Closed
JDGrimes added a commit that referenced this issue May 23, 2016
@JDGrimes
Debug: list all codeception files on Travis
610af9c 
See #92
JDGrimes added a commit that referenced this issue May 24, 2016
@JDGrimes
Correct path to codeception tests
39de5e4 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Correct path
2afb474 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Disable fetching feeds for dashboard wigets
7edc32e 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Set version in the core update transient
5102e95 
It is expected, and if not present can give a notice.

See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Move create points reaction UI test
7ba58b4 
We called it “save”, but “create” is more descriptive.

See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Disable compression testing during the tests
df652ac 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Test that the "Add New" button is disabled
9a55986 
The button should be disabled when we add a new reaction, and then
re-enabled after that reaction is saved.

See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Move disable_* method calls to a separate method
8ae072e 
This is better if somebody wants to extend this class in the future.

See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Move disable_compression_testing()
fed28ef 
Keeps the streamlining methods together.

See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Redirect to the correct page from the login screen
9e9c34f 
This saves time in the tests by cutting out a step.

See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Assert that a button is not disabled
027972f 
Rather than asserting that we don’t have a disabled button. The whole
button being gone would fulfill that requirement.

See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
UI: test canceling creating a points reaction
b56acda 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Improve codeception tests directory structure
800c0d6 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Codeception: allow for custom configs
d152c62 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Correct arg order
39839ea 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Remove unnecessary assertions
7f7342c 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
UI: test canceling updating a points reaction
26e277d 
See #92
JDGrimes added a commit that referenced this issue May 28, 2016
@JDGrimes
Travis CI: fix file name for codeception tests
85d1365 
We’ve been moving things around.

See #92
@JDGrimes JDGrimes mentioned this issue May 30, 2016
Closed
JDGrimes added a commit that referenced this issue May 31, 2016
@JDGrimes
Debug whether this is even the right dir
f3c8ec9 
We run the PHP server in this dir, but I’m now wondering whether it is
correct, because WordPress isn’t being loaded.

See #92
JDGrimes added a commit that referenced this issue Jun 4, 2016
@JDGrimes
Try a different port, run kept tests on 5.6
e1391c4 
More debugging of the acceptance tests.

See #92, #46
JDGrimes added a commit that referenced this issue Jun 13, 2016
@JDGrimes
Try leaving the http in the server URI
fc1de2c 
See #92, #46
JDGrimes added a commit that referenced this issue Jun 13, 2016
@JDGrimes
More "scenario-like" scenario descriptions
5b28184 
See #92
@JDGrimes
Copy link
Member Author

We're still figuring running these on Travis CI out, but we've been tracking that in WordPoints/dev-lib#46 anyway. So we can close this out now.

JDGrimes added a commit that referenced this issue Jun 13, 2016
@JDGrimes
Remove some debugging code
e88cf5e 
See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Jun 13, 2016
@JDGrimes
Set up wp-config.php on Travis CI
0833341 
When running the acceptance tests this is necessary.

See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Jun 14, 2016
@JDGrimes
Update site URL in tests config
483850d 
See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Jun 14, 2016
@JDGrimes
Don't modify site URL before phpunit tests
9d7cf39 
Wait until just before we are running the codeception tests.

See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Jun 14, 2016
@JDGrimes
Remove some debugging code
088b513 
See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Jun 14, 2016
@JDGrimes
Travis: run entire codeception test suite
7585ab9 
We had limited this to a single test for debugging.

See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Jun 14, 2016
@JDGrimes
Travis: Don't use xvfb
c98e19d 
We don’t need to use this to run our acceptance tests when we are using
the PhantomJS browser, so starting it is just a waste of time.

See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Jun 14, 2016
@JDGrimes
Travis: less output during codeception tests
61f0c06 
We wanted all the info we could get for debugging, but now that things
are working, there’s no need.

See #92, WordPoints/dev-lib#46
JDGrimes added a commit that referenced this issue Jun 14, 2016
@JDGrimes
Travis: no more debug for wpcept
43f3105 
We aren’t debugging anymore!

See #92, WordPoints/dev-lib#46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JDGrimes JDGrimes

Labels
enhancement
Projects
None yet
Milestone
0.1.0
Development

No branches or pull requests
1 participant
@JDGrimes