Skip to content
This repository has been archived by the owner on Dec 7, 2018. It is now read-only.

unit test angular #3

Closed
24 tasks done
Ks89 opened this issue Nov 8, 2016 · 4 comments
Closed
24 tasks done

unit test angular #3

Ks89 opened this issue Nov 8, 2016 · 4 comments

Comments

@Ks89
Copy link
Owner

Ks89 commented Nov 8, 2016

  • application
  • carousel (partially done with some small bugs)
  • footer
  • navbar
  • page-header
  • project-search pipe
  • auth service
  • auth-guard service
  • contact service
  • profile service
  • projects service
  • [ ] module and routing (rescheduled for next alpha versions)
  • about
  • activate
  • contact
  • cv
  • forgot
  • home (partially done with some small bugs)
  • login
  • post3d-auth (add button's click test)
  • profile (missing connect/unlink test because they aren't unit tests -> remove some stuff)
  • projectDetail
  • projectlist (partially done with some small bugs - add click's test on a project link)
  • register (same as reset - find a way to click on a navigation link)
  • reset (partially done - find a way to click on a navigation link)
  • [ ] admin SPA (rescheduled for next alpha versions)
@Ks89 Ks89 added this to the alpha-4 milestone Nov 8, 2016
@Ks89 Ks89 self-assigned this Nov 8, 2016
@Ks89 Ks89 changed the title test everything unit test angular Nov 30, 2016
@Ks89
Copy link
Owner Author

Ks89 commented Nov 30, 2016

unit test angular components, directives, services and so on

e2e testing requires a dedicated issue

Ks89 added a commit that referenced this issue Dec 2, 2016
Ks89 added a commit that referenced this issue Dec 3, 2016
Ks89 added a commit that referenced this issue Dec 8, 2016
Ks89 added a commit that referenced this issue Dec 9, 2016
Ks89 added a commit that referenced this issue Dec 9, 2016
Ks89 added a commit that referenced this issue Dec 10, 2016
Ks89 added a commit that referenced this issue Dec 10, 2016
Ks89 added a commit that referenced this issue Dec 10, 2016
Ks89 added a commit that referenced this issue Dec 10, 2016
Ks89 added a commit that referenced this issue Dec 11, 2016
Ks89 added a commit that referenced this issue Dec 11, 2016
Ks89 added a commit that referenced this issue Dec 11, 2016
Ks89 added a commit that referenced this issue Dec 11, 2016
Ks89 added a commit that referenced this issue Dec 12, 2016
Ks89 added a commit that referenced this issue Dec 12, 2016
Ks89 added a commit that referenced this issue Dec 13, 2016
Ks89 added a commit that referenced this issue Dec 19, 2016
Ks89 added a commit that referenced this issue Dec 26, 2016
Ks89 added a commit that referenced this issue Dec 26, 2016
Ks89 added a commit that referenced this issue Dec 27, 2016
Ks89 added a commit that referenced this issue Dec 27, 2016
Ks89 added a commit that referenced this issue Dec 27, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
@Ks89
Copy link
Owner Author

Ks89 commented Dec 28, 2016

80.9% of coverage and I have to do some other tests. I want to use 3 or 4 day to this issue and finally I'll move it to the next one.

In next alphas I'll cover >= 90%, at the moment 80% is enough :)

Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
Ks89 added a commit that referenced this issue Dec 28, 2016
@Ks89
Copy link
Owner Author

Ks89 commented Dec 28, 2016

83.6% is enough

@Ks89
Copy link
Owner Author

Ks89 commented Dec 29, 2016

85.4% OK, now I can close this issue and create two other issues for the two things that I didn't complete

@Ks89 Ks89 closed this as completed Dec 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant