Added @JsfTest and JsfTestRunner information

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11343 761efcf2-d34d-6b61-9145-99dcacc3edf1
Added 2.2.0-m09 as test version

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11346 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixes http://java.net/jira/browse/JAVASERVERFACES-2673, r=rogerk, changes to make sure the getViewMap works correctly during initialization as well

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11347 761efcf2-d34d-6b61-9145-99dcacc3edf1
    http://java.net/jira/browse/JAVASERVERFACES-2581


SECTION: API Changes
----------------------------
M       jsf-api/src/main/java/javax/faces/flow/Flow.java
A       jsf-api/src/main/java/javax/faces/flow/ReturnNode.java

- Introduce ReturnNode type

-    public abstract Map<String,NavigationCase> getReturns();
+    public abstract Map<String,ReturnNode> getReturns();

  This allows the implementation of getNode() to cover all the different node types.

M       jsf-api/src/main/java/javax/faces/flow/builder/MethodCallBuilder.java

- Fix return type of defaultOutcome() to be correct.

- Add expression(String)

M       jsf-api/src/main/java/javax/faces/flow/builder/FlowCallBuilder.java

- remove inboundParameter().  I'll have to add something back in its
  place though, for 2677-FlowXSDTweaks.

M       jsf-api/src/main/java/javax/faces/flow/builder/FlowBuilder.java

- Remove "link()" method.

SECTION: Modified Files
----------------------------
M       jsf-ri/src/main/java/com/sun/faces/facelets/flow/MethodCallTagHandler.java

- Even though this code is going away, it still has to be correct while
  it's here.  Without this change it's not possible to have multiple
  method-call nodes!

M       jsf-ri/src/main/java/com/sun/faces/facelets/flow/FacesFlowDefinitionTagHandler.java
A       jsf-ri/src/main/java/com/sun/faces/flow/ReturnNodeImpl.java

- Account for introduction of ReturnNode type.

M       jsf-ri/src/main/java/com/sun/faces/application/NavigationHandlerImpl.java

- Make method-call work.

M       jsf-ri/src/main/java/com/sun/faces/flow/builder/FlowBuilderImpl.java

- Remove link() method.

M       jsf-ri/src/main/java/com/sun/faces/flow/builder/MethodCallBuilderImpl.java

- Replace UnsupportedOperationExceptions with working code.

M       jsf-ri/src/main/java/com/sun/faces/flow/builder/ReturnBuilderImpl.java

- Account for introduction of ReturnNode type.

M       jsf-ri/src/main/java/com/sun/faces/flow/builder/FlowCallBuilderImpl.java

- remove inboundParameter().  I'll have to add something back in its
  place though, for 2677-FlowXSDTweaks.

M       jsf-ri/src/main/java/com/sun/faces/flow/FlowImpl.java

- Account for introduction of ReturnNode type

M       test/web-profile/flow/pom.xml
A       test/web-profile/flow/basic_method_call
A       test/web-profile/flow/basic_method_call/nbactions.xml
A       test/web-profile/flow/basic_method_call/src
A       test/web-profile/flow/basic_method_call/src/test
A       test/web-profile/flow/basic_method_call/src/test/java
A       test/web-profile/flow/basic_method_call/src/test/java/com
A       test/web-profile/flow/basic_method_call/src/test/java/com/sun
A       test/web-profile/flow/basic_method_call/src/test/java/com/sun/faces
A       test/web-profile/flow/basic_method_call/src/test/java/com/sun/faces/test
A       test/web-profile/flow/basic_method_call/src/test/java/com/sun/faces/test/webprofile
A       test/web-profile/flow/basic_method_call/src/test/java/com/sun/faces/test/webprofile/flow
A       test/web-profile/flow/basic_method_call/src/test/java/com/sun/faces/test/webprofile/flow/basic_method_call
A       test/web-profile/flow/basic_method_call/src/test/java/com/sun/faces/test/webprofile/flow/basic_method_call/FlowMethodCallIT.java
A       test/web-profile/flow/basic_method_call/src/main
A       test/web-profile/flow/basic_method_call/src/main/java
A       test/web-profile/flow/basic_method_call/src/main/java/com
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun/faces
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun/faces/test
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun/faces/test/webprofile
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun/faces/test/webprofile/flow
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun/faces/test/webprofile/flow/basic_method_call
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun/faces/test/webprofile/flow/basic_method_call/FlowA.java
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun/faces/test/webprofile/flow/basic_method_call/Flow_a_Bean.java
A       test/web-profile/flow/basic_method_call/src/main/java/com/sun/faces/test/webprofile/flow/basic_method_call/Flow_b_Bean.java
A       test/web-profile/flow/basic_method_call/src/main/resources
A       test/web-profile/flow/basic_method_call/src/main/resources/META-INF
A       test/web-profile/flow/basic_method_call/src/main/webapp
A       test/web-profile/flow/basic_method_call/src/main/webapp/index.xhtml
A       test/web-profile/flow/basic_method_call/src/main/webapp/flow-a
A       test/web-profile/flow/basic_method_call/src/main/webapp/flow-a/flow-a.xhtml
A       test/web-profile/flow/basic_method_call/src/main/webapp/flow-a/next_b.xhtml
A       test/web-profile/flow/basic_method_call/src/main/webapp/flow-b
A       test/web-profile/flow/basic_method_call/src/main/webapp/flow-b/flow-b.xhtml
A       test/web-profile/flow/basic_method_call/src/main/webapp/flow-b/flow-b-flow.xml
A       test/web-profile/flow/basic_method_call/src/main/webapp/flow-b/next_b.xhtml
A       test/web-profile/flow/basic_method_call/src/main/webapp/WEB-INF
A       test/web-profile/flow/basic_method_call/src/main/webapp/WEB-INF/faces-config.xml
A       test/web-profile/flow/basic_method_call/src/main/webapp/WEB-INF/beans.xml
A       test/web-profile/flow/basic_method_call/src/main/webapp/WEB-INF/web.xml
A       test/web-profile/flow/basic_method_call/src/main/webapp/WEB-INF/glassfish-web.xml
A       test/web-profile/flow/basic_method_call/src/main/webapp/return1.xhtml
A       test/web-profile/flow/basic_method_call/pom.xml

- New test content.


git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11348 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixed PMD issue

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11349 761efcf2-d34d-6b61-9145-99dcacc3edf1
SECTION: Modified Files
----------------------------
M       jsf-ri/src/main/java/com/sun/faces/flow/builder/FlowBuilderImpl.java

- Unused import


git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11350 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixes http://java.net/jira/browse/JAVASERVERFACES-2671, r=rogerk, make sure we ignore a f:view contracts attributes not on the top level.

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11351 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixes http://java.net/jira/browse/JAVASERVERFACES-2641, r=rogerk, make sure the @ViewScoped CDI annotation uses the view map to store instances.

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11354 761efcf2-d34d-6b61-9145-99dcacc3edf1
Disabling test while fixing is underway

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11355 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixes http://java.net/jira/browse/JAVASERVERFACES-2641, make sure we use identityHashCode when using the view map as a key.

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11356 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixed FindBugs issue

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11357 761efcf2-d34d-6b61-9145-99dcacc3edf1
http://java.net/jira/browse/JAVASERVERFACES-2581

- Implement switch portion of FlowBuilder


SECTION: API Changes
----------------------------
A  +    jsf-api/src/main/java/javax/faces/flow/builder/SwitchCaseBuilder.java
D       jsf-api/src/main/java/javax/faces/flow/builder/SwitchCase.java

- Rename this.  Remove unnecessary methods too.

M       jsf-api/src/main/java/javax/faces/flow/builder/ViewBuilder.java
M       jsf-api/src/main/java/javax/faces/flow/builder/MethodCallBuilder.java
M       jsf-api/src/main/java/javax/faces/flow/builder/FlowCallBuilder.java

- No need to re-declare markAsStartNode() since it comes by virtue of
  saying this abstract class implements NodeBuilder.

M       jsf-api/src/main/java/javax/faces/flow/builder/ReturnBuilder.java

- No need to re-declare markAsStartNode() since it comes by virtue of
  saying this abstract class implements NodeBuilder.

- Remove outboundParameter().  Returns don't have outbound parameters,
  per Marta Hawkins.

M       jsf-api/src/main/java/javax/faces/flow/builder/SwitchBuilder.java

- No need to re-declare markAsStartNode() since it comes by virtue of
  saying this abstract class implements NodeBuilder.

- Change the return type of methods based on
  SwitchCase->SwitchCaseBuilder class name change.

M       jsf-api/src/main/java/javax/faces/flow/Flow.java

- Remove unused method getNavigationCases().

SECTION: Impl changes
----------------------------
M       jsf-ri/src/main/java/com/sun/faces/flow/FlowImpl.java
M       jsf-ri/src/main/java/com/sun/faces/facelets/flow/FacesFlowDefinitionTagHandler.java
M       jsf-ri/src/main/java/com/sun/faces/flow/builder/ViewBuilderImpl.java
M       jsf-ri/src/main/java/com/sun/faces/flow/builder/MethodCallBuilderImpl.java
M       jsf-ri/src/main/java/com/sun/faces/flow/builder/ReturnBuilderImpl.java
M       jsf-ri/src/main/java/com/sun/faces/flow/builder/FlowCallBuilderImpl.java

- Honor immutability contracts.

M       jsf-ri/src/main/java/com/sun/faces/facelets/flow/SwitchNodeTagHandler.java
M       jsf-ri/src/main/java/com/sun/faces/facelets/flow/NavigationCaseTagHandler.java

- Fix bug where state was not correctly cleaned up between tag invocations.

M       jsf-ri/src/main/java/com/sun/faces/facelets/flow/FlowNavigationCase.java

- Add setter for conditionExpression property.

M       jsf-ri/src/main/java/com/sun/faces/application/NavigationHandlerImpl.java

- The interaction between the Flow system and the NavigationHendler is
  necessarily intimate.  In this change, I had to make it so the
  population of the internal private Navigation Map from the FlowHandler
  is done in the eager case.  Prior to this change, it was only done in
  the lazy case.  When JAVASERVERFACES-2580 is done, the eager case will
  no longer be necessary and can be removed.

M       jsf-ri/src/main/java/com/sun/faces/flow/builder/FlowBuilderImpl.java

- Make sure to initialize the flow so the switches get correctly populated.

M       jsf-ri/src/main/java/com/sun/faces/flow/builder/SwitchBuilderImpl.java
A       jsf-ri/src/main/java/com/sun/faces/flow/builder/SwitchCaseBuilderImpl.java

- Implement.

M       jsf-ri/src/main/java/com/sun/faces/flow/SwitchNodeImpl.java

- Implement.

- Honor immutability contracts.

M       test/web-profile/flow/pom.xml
A       test/web-profile/flow/basic_switch
A       test/web-profile/flow/basic_switch/nbactions.xml
A       test/web-profile/flow/basic_switch/src
A       test/web-profile/flow/basic_switch/src/test
A       test/web-profile/flow/basic_switch/src/test/java
A       test/web-profile/flow/basic_switch/src/test/java/com
A       test/web-profile/flow/basic_switch/src/test/java/com/sun
A       test/web-profile/flow/basic_switch/src/test/java/com/sun/faces
A       test/web-profile/flow/basic_switch/src/test/java/com/sun/faces/test
A       test/web-profile/flow/basic_switch/src/test/java/com/sun/faces/test/webprofile
A       test/web-profile/flow/basic_switch/src/test/java/com/sun/faces/test/webprofile/flow
A       test/web-profile/flow/basic_switch/src/test/java/com/sun/faces/test/webprofile/flow/basic_switch
A  +    test/web-profile/flow/basic_switch/src/test/java/com/sun/faces/test/webprofile/flow/basic_switch/SwitchCallIT.java
A       test/web-profile/flow/basic_switch/src/main
A       test/web-profile/flow/basic_switch/src/main/java
A       test/web-profile/flow/basic_switch/src/main/java/com
A       test/web-profile/flow/basic_switch/src/main/java/com/sun
A       test/web-profile/flow/basic_switch/src/main/java/com/sun/faces
A       test/web-profile/flow/basic_switch/src/main/java/com/sun/faces/test
A       test/web-profile/flow/basic_switch/src/main/java/com/sun/faces/test/webprofile
A       test/web-profile/flow/basic_switch/src/main/java/com/sun/faces/test/webprofile/flow
A       test/web-profile/flow/basic_switch/src/main/java/com/sun/faces/test/webprofile/flow/basic_switch
A       test/web-profile/flow/basic_switch/src/main/java/com/sun/faces/test/webprofile/flow/basic_switch/FlowA.java
A       test/web-profile/flow/basic_switch/src/main/java/com/sun/faces/test/webprofile/flow/basic_switch/Flow_a_Bean.java
A       test/web-profile/flow/basic_switch/src/main/java/com/sun/faces/test/webprofile/flow/basic_switch/Flow_b_Bean.java
A       test/web-profile/flow/basic_switch/src/main/resources
A       test/web-profile/flow/basic_switch/src/main/resources/META-INF
A       test/web-profile/flow/basic_switch/src/main/webapp
A       test/web-profile/flow/basic_switch/src/main/webapp/index.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-a
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-a/flow-a.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-a/switchA_result.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-a/switchB_result.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-a/switchC_result.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-b
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-b/flow-b.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-b/flow-b-flow.xml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-b/switchA_result.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-b/switchB_result.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/flow-b/switchC_result.xhtml
A       test/web-profile/flow/basic_switch/src/main/webapp/WEB-INF
A       test/web-profile/flow/basic_switch/src/main/webapp/WEB-INF/faces-config.xml
A       test/web-profile/flow/basic_switch/src/main/webapp/WEB-INF/beans.xml
A       test/web-profile/flow/basic_switch/src/main/webapp/WEB-INF/web.xml
A       test/web-profile/flow/basic_switch/src/main/webapp/WEB-INF/glassfish-web.xml
A       test/web-profile/flow/basic_switch/src/main/webapp/return1.xhtml
A       test/web-profile/flow/basic_switch/pom.xml

- New test content.


git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11358 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fix unused imports.

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11359 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixes http://java.net/jira/browse/JAVASERVERFACES-2641, make sure we iterate over all view maps in a session when the session gets destroyed.

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11360 761efcf2-d34d-6b61-9145-99dcacc3edf1
<< http://java.net/jira/browse/JAVASERVERFACES-2648 >>


SECTION: Modified Files
----------------------------
M       jsf-ri/src/main/java/com/sun/faces/context/ExternalContextImpl.java
---- redirect method: for Ajax requests, check if the response is complete, 
---- throw an IllegalArgumentException.  This is handled in the non-Ajax
---- case when srvlet sendRedirect is called.


A       test/agnostic/ajax/src/test/java/com/sun/faces/test/agnostic/ajax/Issue2648IT.java
A       test/agnostic/ajax/src/main/java/com/sun/faces/test/agnostic/ajax/Issue2648Bean.java
A       test/agnostic/ajax/src/main/webapp/issue2648-1.xhtml
A       test/agnostic/ajax/src/main/webapp/issue2648-2.xhtml
A       test/agnostic/ajax/src/main/webapp/issue2648.xhtml
---- Test Case.




git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11361 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixed PMD issue, remove unused import

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11363 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixes http://java.net/jira/browse/JAVASERVERFACES-2681, migrating JAVASERVERFACES-2347 to new test harness.

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11365 761efcf2-d34d-6b61-9145-99dcacc3edf1
Ignore test while investigating why if fails on clientSide state saving after migration to new harness

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11367 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixes http://java.net/jira/browse/JAVASERVERFACES-2685, migrate JAVASERVERFACES-2258 to new test harness.

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11372 761efcf2-d34d-6b61-9145-99dcacc3edf1
Fixes http://java.net/jira/browse/JAVASERVERFACES-2686, migrate JAVASERVERFACES-2111 to new test harness.

git-svn-id: https://svn.java.net/svn/mojarra~svn/trunk@11376 761efcf2-d34d-6b61-9145-99dcacc3edf1