Skip to content

Commit

Permalink
Clean up old rabbitmq support
Browse files Browse the repository at this point in the history
- Only scan for rabbit service label "rabbitmq-2.4"

- Continue to parse credentials in both old (broken apart)
 and new (url) formats

Change-Id: Ie9fc198232b1f88ecee9ff29b7ed70cc84815b85
  • Loading branch information
Jennifer Hickey committed Sep 26, 2011
1 parent eb5d64d commit 121c067
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 8 deletions.
Expand Up @@ -48,13 +48,7 @@ private static void labelledServiceType(Class<? extends AbstractServiceInfo> ser
labelledServiceType(RdbmsServiceInfo.class, "postgresql-9.0");
labelledServiceType(RedisServiceInfo.class, "redis-2.2");
labelledServiceType(MongoServiceInfo.class, "mongodb-1.8");

// the old rabbitmq service
labelledServiceType(RabbitServiceInfo.class, "rabbitmq-2.4");
// rabbitmq SRS-based service, during testing
labelledServiceType(RabbitServiceInfo.class, "rabbitmq-srs-2.4.1");
// rabbitmq SRS-based service, after the old service is gone
labelledServiceType(RabbitServiceInfo.class, "rabbitmq-2.4.1");
}

/* package for testing purpose */
Expand Down
Expand Up @@ -116,6 +116,10 @@ public void getServiceInfoPostgresql() {
assertEquals(password, info.getPassword());
}

/**
* Verifies parsing of credentials in the format of "old"
* Rabbit service (still used by chef recipe deployment)
*/
@Test
public void getServiceInfoRabbit() {
String serviceName = "rabbit-1";
Expand All @@ -133,6 +137,9 @@ public void getServiceInfoRabbit() {
assertEquals(port, info.getPort());
}

/**
* Verifies parsing of credentials in the existing Rabbit service
*/
@Test
public void getServiceSRSInfoRabbit() {
String serviceName = "rabbit-1";
Expand Down
Expand Up @@ -2,6 +2,7 @@
"name":"$serviceName",
"label":"rabbitmq-2.4",
"plan":"free",
"tags":["rabbitmq"],
"credentials":{
"hostname":"$hostname",
"port":$port,
Expand Down
@@ -1,8 +1,8 @@
{
"name":"$serviceName",
"label":"rabbitmq-srs-2.4.1",
"label":"rabbitmq-2.4",
"plan":"free",
"tags":["rabbitmq-srs"],
"tags":["rabbitmq"],
"credentials":{
"url":"$url"
}
Expand Down

0 comments on commit 121c067

Please sign in to comment.