Skip to content
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

move test Uuid creation functions into new module #182

Merged
merged 2 commits into from Mar 25, 2018
Merged

move test Uuid creation functions into new module #182

merged 2 commits into from Mar 25, 2018

Conversation

kinggoesgaming
Copy link
Member

I'm submitting a ...

  • bug fix
  • feature enhancement
  • deprecation or removal
  • refactor

Description

Refactor the tests::new and tests::new2 functions into test_util module.

Motivation

Part of the great refactor. Allows different tests in different modules access the test constructors for Uuid

Tests

The tests are passing

Related Issue(s)

#124

Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
@Dylan-DPC-zz
Copy link
Member

bors: r+

bors bot added a commit that referenced this pull request Mar 25, 2018
182: move test Uuid creation functions into new module r=Dylan-DPC a=kinggoesgaming

**I'm submitting a ...**
  - [ ] bug fix
  - [ ] feature enhancement
  - [ ] deprecation or removal
  - [x] refactor

# Description
Refactor the `tests::new` and `tests::new2` functions into `test_util` module.

# Motivation
Part of the great refactor. Allows different tests in different modules access the test constructors for `Uuid`

# Tests
The tests are passing

# Related Issue(s)
#124
@bors
Copy link
Contributor

bors bot commented Mar 25, 2018

@bors bors bot merged commit e53b922 into master Mar 25, 2018
@kinggoesgaming kinggoesgaming deleted the test_utils branch March 25, 2018 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants