feat: expose all the methods of notification

[athakor]

Fixes #138

This PR exposing all the methods of notifications, also contain the unit test as well as system test of each method.

[codecov]

Codecov Report

Merging #141 into master will increase coverage by 0.48%.
The diff coverage is 80.55%.

@@             Coverage Diff              @@
##             master     #141      +/-   ##
============================================
+ Coverage     63.13%   63.61%   +0.48%     
- Complexity      601      632      +31     
============================================
  Files            32       33       +1     
  Lines          5078     5222     +144     
  Branches        481      498      +17     
============================================
+ Hits           3206     3322     +116     
- Misses         1708     1730      +22     
- Partials        164      170       +6     

Impacted Files	Coverage Δ	Complexity Δ
...rc/main/java/com/google/cloud/storage/Storage.java	80.90% <ø> (ø)	0.00 <0.00> (ø)
...om/google/cloud/storage/spi/v1/HttpStorageRpc.java	1.50% <0.00%> (-0.02%)	1.00 <0.00> (ø)
...ogle/cloud/storage/spi/v1/HttpStorageRpcSpans.java	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
...va/com/google/cloud/storage/spi/v1/StorageRpc.java	64.86% <ø> (ø)	0.00 <0.00> (ø)
...ain/java/com/google/cloud/storage/StorageImpl.java	80.24% <72.41%> (-0.29%)	135.00 <4.00> (+4.00)	⬇️
...ava/com/google/cloud/storage/NotificationInfo.java	90.38% <90.38%> (ø)	26.00 <26.00> (?)
...ogle/cloud/storage/testing/StorageRpcTestBase.java	98.00% <100.00%> (+0.04%)	49.00 <1.00> (+1.00)

---

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 34d1633...300c604. Read the comment docs.

[frankyn]

Please don't rely on PubSub in Notification support.

[frankyn]

Don't rely on pubsub in this case please.

[athakor]

done

[frankyn]

There's a linkage issue:

SEVERE: Newly introduced problem:
(google-cloud-nio-0.120.0-alpha.jar) com.google.cloud.storage.contrib.nio.testing.FakeStorageRpc's method getNotification(String arg1, String arg2) is not implemented in the class
  referenced from com.google.cloud.storage.spi.v1.StorageRpc (google-cloud-storage-1.104.0.jar)

[athakor]

@frankyn here we cant do anything to fix this issue,we can fix it in java-storage-nio. I have already raised the issue over there you can see here storage-nio-38.

[athakor]

@frankyn here, how we can move ahead with above situation?

[frankyn]

Discussing this now in #179, looks like we've been making interface breaking changers and not bumping java major version. Apologies for the delay @athakor

[athakor]

@frankyn Updated branch as per 241 PTAL when you get chance.

[frankyn]

@athakor could you merge master and resolve conflicts?

[athakor]

@athakor could you merge master and resolve conflicts?

@frankyn done. still there's a linkage issue.

[frankyn]

Thanks, we need one more release of the libraries-bom. Should be coming soon and wanted to prep to have your pr ready.

[athakor]

@frankyn thanks for your quick response.

[frankyn]

whoops, didn't mean to close. libraries-bom released and rerunning tests.

[frankyn]

LGTM