Skip to content

Add parameterized test for from_rfc3339 with nanos#7675

Merged
crwilcox merged 2 commits intogoogleapis:masterfrom
crwilcox:add-parameterized-datetime-nano-test
May 20, 2019
Merged

Add parameterized test for from_rfc3339 with nanos#7675
crwilcox merged 2 commits intogoogleapis:masterfrom
crwilcox:add-parameterized-datetime-nano-test

Conversation

@crwilcox
Copy link
Copy Markdown
Contributor

@crwilcox crwilcox commented Apr 8, 2019

While looking into another issue I noticed we didn't seem to test this constructor much (if at all).

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 8, 2019
@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Apr 15, 2019
Copy link
Copy Markdown
Contributor

@tseaver tseaver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The WebRisk failure seems clearly unrelated. However, the API core failures (lint, pytype) need to be fixed.

@crwilcox crwilcox added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 15, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 15, 2019
@crwilcox crwilcox added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 20, 2019
@yoshi-kokoro yoshi-kokoro removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels May 20, 2019
@crwilcox
Copy link
Copy Markdown
Contributor Author

The failures are unrelated. This is a test only change. Merging

@crwilcox crwilcox merged commit ffc97ed into googleapis:master May 20, 2019
@tseaver
Copy link
Copy Markdown
Contributor

tseaver commented May 20, 2019

@crwilcox We still need to fix the pytype failure, or else disable that test.

parthea pushed a commit that referenced this pull request Nov 24, 2025
* Add parameterized test for from_rfc3339 with nanos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. 🚨 This issue needs some love.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants