-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Feature request: add unit tests #3
Comments
Totally -- PR's would be very appreciated for this as I'm swamped for the next month or so. Anyone out there wanna contribute some sweet unit tests? :) |
Ok, I could try but I guess it will take some time because I don't have much experience writing Angular2 unit tests. I will follow the official guide. |
Okay -- we might have to do 1-2 rounds of revisions FYI, just to ensure |
I have sent you an email. |
I'd be happy to write some tests. Where should I start? |
@pauldubois777 awesome! we're currently chatting about this in our private slack channel; can you send me an email at [email protected] and I'll invite you? |
Email sent. Thanks! |
This thread sort of when cold, any progress on the unit tests? Please let me know if anyone needs more help on this task. |
@babilog yeah, this ended up not happening the first time around unf. If you make a PR for unit tests I will absolutely accept them 👍 |
any progress on the unit test? |
Hi @umondelo , unfortunately I'm still working on these at the moment. I can probably push a branch later this weekend. |
@EricSimons , I'm trying to push my local branch to the repo but currently receiving a permissions issue. Running |
|
@babilog only those added to the collaborators can make changes directly to this repo. You need to create a PR to get the changes pushed here after someone verifies them. To do so upload the repo to your account and create a PR with your new repo to this one. |
@babilog ^ what @SandeeshS said! Feel free to make a pull request & I'll review + accept 👍 |
@babilog do you have the tests? I would be grateful for them 🥇 |
any update on unit tests?! |
Feature/add auth to proxy conf * added /auth/ path proxy resolving to proxy.conf.json.
Without unit tests, this project isn't world class project. This project promotes really bad practice and doesn't inform users about that. In modern programming, absence of unit testing disqualifies a project from production workloads. This isn't good project. |
It would be nice to add unit tests for this app.
The text was updated successfully, but these errors were encountered: