Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

fix: allow dots and colons in project id#36

Merged
olavloite merged 2 commits intomasterfrom
issue-33
Jan 22, 2020
Merged

fix: allow dots and colons in project id#36
olavloite merged 2 commits intomasterfrom
issue-33

Conversation

@olavloite
Copy link
Copy Markdown
Collaborator

The project ID part of a connection URL should allow the use of dots and colons. This PR also replaces all usages of the deprecated class org.junit.Assert in the affected test classes with com.google.common.truth.Truth.

Fixes #33

The project ID part of a connection URL should allow the use of dots
and colons.

Fixes #33
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 20, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 20, 2020

Codecov Report

Merging #36 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #36   +/-   ##
=========================================
  Coverage     77.79%   77.79%           
  Complexity     1713     1713           
=========================================
  Files            60       60           
  Lines          5854     5854           
  Branches        819      819           
=========================================
  Hits           4554     4554           
  Misses          926      926           
  Partials        374      374
Impacted Files Coverage Δ Complexity Δ
...m/google/cloud/spanner/jdbc/ConnectionOptions.java 84.24% <ø> (ø) 46 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 769fba6...6dc208b. Read the comment docs.

Comment thread src/main/java/com/google/cloud/spanner/jdbc/ConnectionOptions.java Outdated
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CloudSpanner JDBC: Project IDs that contain period and colon fail URL matching when creating connections

3 participants