[jboss-dev-forums] [Design of Clustering on JBoss (Clusters/JBoss)] - Re: JBAS-4228 Preferred master node for HASingleton

[galder.zamarreno@jboss.com]

"bstansberry at jboss.com" wrote : I'm not looking at code right now, just using slightly tired memory of what I saw last week, so please bear with me...
  | 
  | anonymous wrote : Re 1) Each of the different preferred master definitions is in a different test method, i.e. testIpPort, testHostPort, testOnlyPort, testNonNumericPort, testUnknownHost... that should be enough, shouldn't it? 
  | 
  | I was tallking about the old test method that's been there all along, where you'd added some assertions re: some beans you'd added that use preferred master.  I was just wanted those assertions moved into a separate method, so that the original method was a test of HASingletonElectionPolicySimple and the other assertions were in a separate method.  Just a minor thing.  

Apologies if my previous explanation was not precise enough. In fact, I've created a separate test class called PreferredMasterElectionPolicyUnitTestCase that contains these tests for two reasons:

1.- They're standalone unit tests, no need to have AS running for them.
2.- They're PreferredMasterElectionPolicy specific in the sense that I test any type of input into setPreferredMaster() and see whether it choses the given master after calling elect(). If the given master is invalid for any reason, it should delegate on the simple election policy.

"bstansberry at jboss.com" wrote : anonymous wrote : Btw, while running the the clustered tests for ha singleton election, I have spotted the following WARN message 
  | 
  | Kudos to you for looking at the logs.  Looking at testsuite logs for unexpected WARNs and ERRORs is an excellent way to find all sorts of random problems no one had thought about.  Tedious and painful but very important to do from time to time.
  | 
  | Your solution sounds fine. 

Cheers :)))

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4128288#4128288

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4128288