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

feat(autocomplete): Add AutocompleteSessionToken Mocks #500

Merged
merged 4 commits into from
Sep 13, 2023
Merged

feat(autocomplete): Add AutocompleteSessionToken Mocks #500

merged 4 commits into from
Sep 13, 2023

Conversation

jessedoyle
Copy link
Contributor

@jessedoyle jessedoyle commented Aug 8, 2023

Hi there - thanks for making a great library!

Previously the google.maps.places.AutocompleteSessionToken class was not mocked by this library.

Developers implementing custom autocomplete operations would likely encounter the following error message in their tests:

TypeError: google.maps.places.AutocompleteSessionToken is not a constructor

This PR adds a mock and exports it for the AutocompleteSessionToken class. The constructor accepts no parameters, and the class does not expose any public functions/methods (docs).

resolves #495


Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • [] Appropriate docs were updated (if necessary)

@jessedoyle jessedoyle changed the title feat(autocpmplete): Add AutocompleteSessionToken Mocks feat(autocomplete): Add AutocompleteSessionToken Mocks Aug 8, 2023
Previously the `google.maps.places.AutocompleteSessionToken` class
was not mocked by this library.

Developers implementing custom autocomplete operations would likely
encounter the following error message in their tests:

```
TypeError: google.maps.places.AutocompleteSessionToken is not a constructor
```

This commit adds a mock and exports it for the `AutocompleteSessionToken`
class. The constructor accepts no parameters, and the class does not
expose any public functions/methods.

resolves: #495
@wangela
Copy link
Member

wangela commented Aug 14, 2023

Thanks for fixing this, @jessedoyle! Please sync with main and we'll be able to merge this.

@wangela wangela self-assigned this Aug 14, 2023
@jessedoyle
Copy link
Contributor Author

Hey @wangela - I've merged main into this branch. Thanks!

@usefulthink usefulthink enabled auto-merge (squash) September 13, 2023 11:07
@usefulthink usefulthink merged commit a59d1d8 into googlemaps:main Sep 13, 2023
github-actions bot pushed a commit that referenced this pull request Sep 13, 2023
Previously the `google.maps.places.AutocompleteSessionToken` class
was not mocked by this library.

Developers implementing custom autocomplete operations would likely
encounter the following error message in their tests:

```
TypeError: google.maps.places.AutocompleteSessionToken is not a constructor
```

This commit adds a mock and exports it for the `AutocompleteSessionToken`
class. The constructor accepts no parameters, and the class does not
expose any public functions/methods.

resolves: #495 a59d1d8
googlemaps-bot pushed a commit that referenced this pull request Sep 13, 2023
## [2.20.0](v2.19.4...v2.20.0) (2023-09-13)

### Features

* **autocomplete:** Add AutocompleteSessionToken ([#500](#500)) ([a59d1d8](a59d1d8)), closes [#495](#495)

### Miscellaneous Chores

* **deps-dev:** bump @rollup/plugin-typescript from 11.1.2 to 11.1.3 ([#511](#511)) ([fb9a1a8](fb9a1a8))
@googlemaps-bot
Copy link
Contributor

🎉 This PR is included in version 2.20.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jessedoyle jessedoyle deleted the feat/autocomplete-session-token branch September 13, 2023 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Looking for a mock for google.maps.places.AutocompleteSessionToken
4 participants