testing this lib about angular-hint HOT 7 CLOSED

@btford I'm trying to set up initial tests 14bb413 but I'm having trouble wiring things together I believe because of the order in which bootstrapping occurs? Any advice? Should I be focusing on E2E tests instead?

from angular-hint.

We talked in-person, but for posterity: individual modules should have unit tests, this repo should have mostly (maybe entirely) E2E tests.

from angular-hint.

This is resolved now, right?

from angular-hint.

@ealtenho can we run some very simple E2E tests with protractor?

from angular-hint.

Yes @btford in the commit e1e7cdb I just pushed I added basic protractor tests . I think these tests will be much less irregular when angularHintLog creates similarly formatted output for all modules (currently some use console.log, some console.group, some console.groupCollapsed).

I am also having a strange issue with my ng-hint default test (commented out in this commit). When I run protractor it does not seem to resolve the route for /allHint.

What is the next step to get continuous tests running?

from angular-hint.

Update: e2e tests in /e2e cover all variations of loading modules with ng-hint-include, ng-hint-exclude, ng-hint or no hinting at all.

We also plan to have a sample application that demonstrates the behavior of AngularHint in an application context.

from angular-hint.

There are now passing e2e tests that test the default logging of AngularHint to the console from each of the AngularHint modules. Additionally, the sample applications example and correctExample showcase the function of AngularHint in a realistic setting. This uncovered the bugs described in #36.

from angular-hint.