Closed hmichal-dev closed 4 years ago
Travis test failed due to incorrect configuration
Nice, I thought previously in adding a feature like this.
Could you please add a test to check if it works as expected?
@elboletaire I can't write JavaScript tests. I'm a backend developer.
I don't understand that argument. I'm a backend developer and I test both backend and frontend, and they are tested the same way. Actually, the tests run in this project are run exactly like any backend tests, but using libraries to simulate there's a DOM.
Just take a look to the test file, and also to this part of the readme where you can see how tests are executed: https://github.com/elboletaire/password-strength-meter#testing
BTW, if you really think you cannot do it, let me know and I'll do it for you when I can.
I test backend but I don't know any framework for testing JavaScript. I'm busy in this month so I can't spend time learning it.
refs #11
Do we still need a test for this? I'd be happy to help.
Thanks @jhoover4, but I'm refactoring entirely. That's why I've tagged it for the version 2.0.0, because it'll bring breaking changes:
min characters
or the contains username field text
errors).Also, for v2 there are other issues on track: https://github.com/elboletaire/password-strength-meter/issues?q=is%3Aopen+is%3Aissue+milestone%3A2.0.0
For sure, makes sense. I guess I was just offering to help with the overall refactoring if you need it! If not, no worries.
Ofc you can do it if you take all that in mind. I'm probably not gonna work on it until next weekend or so; you're free to take any of the issues for the V2 (even for V3 if you want) and make a PR.
Sweet, I'll try to push up this and the other issue I commented on.
Okay I added a PR for this #26
Superseeded by #26
You can define custom steps in options. Steps must be sorted ascending.