Skip to content

Commit 89ed706

Browse files
committed
RSCBC-147: Fix http tests when running against cluster_run
Fix test_json_block_read in tests/http.rs to use the http port, not just assume 8091.
1 parent f4692c4 commit 89ed706

File tree

2 files changed

+14
-6
lines changed

2 files changed

+14
-6
lines changed

sdk/couchbase-core/tests/common/test_config.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ pub struct TestSetupConfig {
4848
pub username: String,
4949
pub password: String,
5050
pub memd_addrs: Vec<Address>,
51+
pub http_addrs: Vec<Address>,
5152
pub data_timeout: String,
5253
pub use_ssl: bool,
5354
pub bucket: String,
@@ -166,6 +167,7 @@ pub async fn create_test_config(test_config: &EnvTestConfig) -> TestSetupConfig
166167
username: test_config.username.clone(),
167168
password: test_config.password.clone(),
168169
memd_addrs: resolved_conn_spec.memd_hosts.clone(),
170+
http_addrs: resolved_conn_spec.http_hosts.clone(),
169171
data_timeout: test_config.data_timeout.clone(),
170172
use_ssl: resolved_conn_spec.use_ssl,
171173
bucket: test_config.default_bucket.clone(),

sdk/couchbase-core/tests/http.rs

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -47,14 +47,17 @@ pub struct QueryMetrics {
4747
#[test]
4848
fn test_row_streamer() {
4949
setup_test(async |config| {
50-
let addrs = config.memd_addrs;
50+
let scheme = if config.use_ssl { "https" } else { "http" };
51+
let addrs = config.http_addrs;
5152

52-
let host = addrs.first().unwrap().host.clone();
53+
let addr = addrs.first().unwrap();
54+
let host = &addr.host;
55+
let port = addr.port;
5356

5457
let basic_auth = BasicAuth::new(config.username, config.password);
5558

5659
let request_body = json!({"statement": "select i from array_range(1, 10000) AS i;"});
57-
let uri = format!("http://{host}:8093/query/service");
60+
let uri = format!("{scheme}://{host}:{port}/_p/query/query/service");
5861

5962
let request = Request::new(Method::POST, uri)
6063
.user_agent("rscbcorex".to_string())
@@ -116,12 +119,15 @@ fn test_row_streamer() {
116119
#[test]
117120
fn test_json_block_read() {
118121
setup_test(async |config| {
119-
let addrs = config.memd_addrs;
122+
let scheme = if config.use_ssl { "https" } else { "http" };
123+
let addrs = config.http_addrs;
120124

121-
let host = addrs.first().unwrap().host.clone();
125+
let addr = addrs.first().unwrap();
126+
let host = &addr.host;
127+
let port = addr.port;
122128

123129
let basic_auth = BasicAuth::new(config.username, config.password);
124-
let uri = format!("http://{host}:8091/pools/default/terseClusterInfo");
130+
let uri = format!("{scheme}://{host}:{port}/pools/default/terseClusterInfo");
125131

126132
let request = Request::new(Method::GET, uri)
127133
.user_agent("rscbcorex".to_string())

0 commit comments

Comments
 (0)