Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: some unit tests for some more utility functions. #12

Conversation

yesudeep
Copy link

@yesudeep yesudeep commented Jul 6, 2023

Moves 2 more functions into separate modules so that we can add unit tests
for them.

  • getLabel is now in string.js and unit tests for it are found in string.test.js
  • mapBetween is now in math.js and unit tests for it are found in math.test.js
  • Note that the calculation for mapBetween has not been changed and the function
    probably needs to be updated to include validation.

@yesudeep yesudeep force-pushed the feat-some-tests-for-utility-functions branch from 5b7438a to 924a30a Compare July 6, 2023 15:24
@yesudeep yesudeep force-pushed the feat-some-tests-for-utility-functions branch from 924a30a to 110d199 Compare July 6, 2023 15:24
@yjmg yjmg merged commit 1ccc019 into looker-open-source:master Jul 6, 2023
jeremytchang pushed a commit that referenced this pull request Jan 3, 2024
🤖 I have created a release *beep* *boop*
---


## 1.0.0 (2024-01-03)


### Features

* add jest for unit testing
([#11](#11))
([76a5e49](76a5e49))
* some unit tests for some more utility functions.
([#12](#12))
([1ccc019](1ccc019))


### Bug Fixes

* Removes trailing and leading special chars from value dimension label
([#10](#10))
([4b50f5a](4b50f5a))
* Viz not updating after filter added or changed
([2c0e133](2c0e133))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants