-
Notifications
You must be signed in to change notification settings - Fork 32
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
Failing Tests Out of the Box on Lucee #95
Comments
I just realized that the tests all pass in ACF9, but not in Lucee 4. That gives me something to work with. |
Jamie- I have yet to actually setup the tests since I started working on this project - I just sort of setup the project and started working on documentation but didn't get to even trying the automated tests. So you are already ahead of me on trying out the tests. Looks like you are making some progress though! -Cameron |
In #94, I've now got all the functional tests working in Lucee 4, and four out of eight previously failing unit tests are now passing. (Four unit tests left.) By the way, assuming I'm doing it right, testing is easy, if you've got the right stuff:
* I have a pull request (teamcfadvance/CFSelenium#25) in for CFSelenium, too, which gets things working there. Once the chips settle, I'll write this up in the readme, or something. |
I'm writing some new features and I'd like to have code coverage for the new bits, as well as prevent regressions in the old bits, but there are a handful of failures right out of the box.
I took a look at a couple of them, but they weren't the sorts of things where it was clear what the intent was, so it wasn't easy to see if the problems were with the tests or with the code under test.
(Note, I'm running with #94; otherwise, I wouldn't be able to run the tests at all.)
Functional Tests
http://127.0.0.1:40878/ValidateThis/functionalTests/run.cfm
Unit Tests
http://127.0.0.1:40878/ValidateThis/unitTests/run.cfm
I realize this isn't the most active project in the world, but if anyone can help me at least start with a firm test foundation, I'd be grateful.
The text was updated successfully, but these errors were encountered: