Remove HTTP influxDbServiceUrl() and HTTP::databaseName() methods (#93)

offa · Jan 28, 2023 · 57b6c42 · 57b6c42
1 parent 4f1f45e
commit 57b6c42
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 34 deletions.
diff --git a/src/HTTP.cxx b/src/HTTP.cxx
@@ -119,16 +119,6 @@ namespace influxdb::transports
  checkResponse(response);
  }
 
- std::string HTTP::databaseName() const
- {
- return mDatabaseName;
- }
-
- std::string HTTP::influxDbServiceUrl() const
- {
- return mInfluxDbServiceUrl;
- }
-
  void HTTP::setProxy(const Proxy& proxy)
  {
  session.SetProxies(cpr::Proxies{{"http", proxy.getProxy()}, {"https", proxy.getProxy()}});

diff --git a/src/HTTP.h b/src/HTTP.h
@@ -63,12 +63,6 @@ namespace influxdb::transports
  /// \param auth <username>:<password>
  void enableBasicAuth(const std::string& auth);
 
- /// Get the database name managed by this transport
- [[nodiscard]] std::string databaseName() const;
-
- /// Get the influx service url which transport connects to
- [[nodiscard]] std::string influxDbServiceUrl() const;
-
  /// Sets proxy
  void setProxy(const Proxy& proxy) override;
 

diff --git a/test/HttpTest.cxx b/test/HttpTest.cxx
@@ -225,24 +225,6 @@ namespace influxdb::test
  http.enableBasicAuth("user0:pass0");
  }
 
- TEST_CASE("Database name is returned if valid", "[HttpTest]")
- {
- ALLOW_CALL(sessionMock, SetTimeout(_));
- ALLOW_CALL(sessionMock, SetConnectTimeout(_));
-
- const HTTP http{"http://localhost:8086?db=example-database-0"};
- CHECK(http.databaseName() == "example-database-0");
- }
-
- TEST_CASE("Database service url is returned if valid", "[HttpTest]")
- {
- ALLOW_CALL(sessionMock, SetTimeout(_));
- ALLOW_CALL(sessionMock, SetConnectTimeout(_));
-
- const HTTP http{"http://localhost:8086?db=example-database-1"};
- CHECK(http.influxDbServiceUrl() == "http://localhost:8086");
- }
-
  TEST_CASE("Set proxy without authentication", "[HttpTest]")
  {
  auto http = createHttp();