Skip to content
Snippets Groups Projects

Reduce the token generator test to 1s for the whole suite

Description

Reduces the number of tokens generated in the token generator test to speed up tests.

Changes

  • The number of tokens is no longer inversely proportional to 100000016, but rather to 2000016, an 128x "improvement" (we already knew pretty sure that the generator was outputting unique tokens)

Additions

None.

Test and Review

To be filled in by the reviewers

  • All of the methods are commented to expectation

  • The methods are tested to satisfaction

  • There are no unnecessary files present in the MR

  • The continuous integration has no problems with the MR

  • The MR is filled in as requested (including labels, milestones, and reviewers)

  • The documentation is up-to-date

  • All nullable parameters are marked as such

Edited by Luc Everse

Merge request reports

Pipeline #174340 passed

Pipeline passed for d5fdc144 on short-token-test

Merged by Tim van der HorstTim van der Horst May 27, 2019 (May 27, 2019 5:52pm UTC)

Loading

Pipeline #174710 passed

Pipeline passed for 3d349b8c on development

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading