Skip to content
This repository has been archived by the owner on Jun 29, 2021. It is now read-only.

Split tests & add some #347

Merged
merged 1 commit into from Jul 29, 2019
Merged

Conversation

hasezoey
Copy link
Contributor

@hasezoey hasezoey commented Jul 24, 2019

TL;DR: split tests, added tests, added test/ to tslint checks

  • remove comments generated from vscode (in .vscode/)
  • adding tslint-eslint-rules to enforce indent(on save)
  • remove "@Format" and apply it for every file for prettier
  • fix wrong indent in some test files
  • remove not needed reconnect
  • include tests/ for tslint & vscode to check
  • extra tsconfig for build
  • normal tsconfig includes test/ now (for tslint & compiler check & vscode)
  • set vscode to indent 2, instead of global
  • small style changes
  • modify Improvemnts in README
  • setting tsconfig target to es6, because node 8+ (set as minimum) support full es6 (and smaller and more readable output)

  • moving "should add" tests to their own file
  • moving "select" tests to their own file
  • moving "string Validator" tests to their own file
  • moving "biguser" test to its own file
  • moving "getClassForDocument" to its own file
  • moved hooktests/* & renamed them

  • adding "minlength" to the stringValidator tests

@coveralls
Copy link

coveralls commented Jul 24, 2019

Coverage Status

Coverage increased (+1.7%) to 86.923% when pulling 0afcdd8 on hasezoey:feature/split_tests into ea1c0ec on szokodiakos:master.

@hasezoey hasezoey force-pushed the feature/split_tests branch 2 times, most recently from 1652c4c to 9c2950c Compare July 25, 2019 15:00
@hasezoey hasezoey marked this pull request as ready for review July 25, 2019 15:02
@hasezoey hasezoey changed the title Feature/split tests Split tests & add some Jul 25, 2019
- remove comments generated from vscode (in .vscode/)
- adding tslint-eslint-rules to enforce indent(on save)
- remove "@Format" and apply it for every file for prettier
- fix wrong indent in some test files
- remove not needed reconnect
- include tests/ for tslint & vscode to check
- extra tsconfig for build that excludes test/
- set vscode to indent 2, instead of global
- small style changes

- moving "should add" tests to their own file
- moving "select" tests to their own file
- moving "string Validator" tests to their own file
- moving "biguser" test to its own file
- moving "getClassForDocument" to its own file

- modify Improvemnts in README
- moved hooktests/* & renamed them
- adding "minlength" to the stringValidator tests
@hasezoey hasezoey mentioned this pull request Jul 28, 2019
@Ben305 Ben305 merged commit 0afcdd8 into szokodiakos:master Jul 29, 2019
@hasezoey hasezoey deleted the feature/split_tests branch July 29, 2019 10:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants