Changeset 1487

Show
Ignore:
Timestamp:
03-02-11 10:51:16 (3 years ago)
Author:
robert@…
Message:

- Fixed StudySearch? unit tests (#303)
- Changed css error message to class="errormessage" in order to avoid breaking the study wizard (see r1485)

Location:
trunk
Files:
5 modified

Legend:

Unmodified
Added
Removed
  • trunk/grails-app/views/advancedQuery/index.gsp

    r1482 r1487  
    4040 
    4141<g:if test="${flash.error}"> 
    42         <div class="error"> 
     42        <div class="errormessage"> 
    4343                ${flash.error.toString().encodeAsHTML()} 
    4444        </div> 
  • trunk/src/groovy/dbnp/query/Search.groovy

    r1483 r1487  
    4949        public void setResultFields( Map r ) { resultFields = r; } 
    5050 
     51        public Search() { 
     52                def ctx = ApplicationHolder.getApplication().getMainContext(); 
     53                def authenticationService = ctx.getBean("authenticationService"); 
     54                def sessionUser = authenticationService?.getLoggedInUser(); 
     55 
     56                if( sessionUser ) 
     57                        this.user = sessionUser; 
     58                else 
     59                        this.user = null 
     60        } 
     61         
    5162        /** 
    5263         * Returns the number of results found by this search 
     
    215226                        return []; 
    216227                         
    217                 // Determine the moduleCommunicationService 
    218                 def ctx = (ApplicationContext)ApplicationHolder.getApplication().getMainContext(); 
     228                // Determine the moduleCommunicationService. Because this object 
     229                // is mocked in the tests, it can't be converted to a ApplicationContext object 
     230                def ctx = ApplicationHolder.getApplication().getMainContext(); 
    219231                def moduleCommunicationService = ctx.getBean("moduleCommunicationService"); 
    220232                         
  • trunk/src/groovy/dbnp/query/StudySearch.groovy

    r1482 r1487  
    6767 
    6868                def studies = Study.list().findAll { it.canRead( this.user ) }; 
    69                  
     69 
    7070                // If no criteria are found, return all studies 
    7171                if( !criteria || criteria.size() == 0 ) { 
  • trunk/test/unit/dbnp/query/StudySearchTests.groovy

    r1478 r1487  
    55import org.dbnp.gdt.AssayModule 
    66import org.codehaus.groovy.grails.commons.ApplicationHolder 
     7import dbnp.authentication.*; 
    78 
    89/** 
     
    3334                ] 
    3435                def assays = [ new Assay( name: 'Assay 1', parent: studies[0], samples: [samples[0]] ), new Assay( name: 'Assay 2', parent: studies[1], samples: [samples[1]] ) ] 
     36 
     37                def users = [ new SecUser( username: "abc" ) ]; 
    3538                 
    3639                mockDomain( Study, studies ); 
     
    4245 
    4346        mockDomain( AssayModule ); 
     47                mockDomain( SecUser, users ) 
     48                mockDomain( SecUserSecRole ) 
     49                mockDomain( SecRole ) 
    4450                 
    4551                subjects.each { it.parent.addToSubjects( it ); } 
     
    4955                samples.each { it.parent.addToSamples( it ); } 
    5056                assays.each { it.parent.addToAssays( it ); } 
     57                studies.each { 
     58                        it.owner = users[0] 
     59                } 
    5160 
    5261        // some mocks to make sure test doesn't break on finding 'moduleCommunicationService' 
    53         ApplicationHolder.metaClass.static.getApplication = { [getMainContext: { [getBean: {a -> null}] }] } 
     62        ApplicationHolder.metaClass.static.getApplication = { [getMainContext:  
     63                        { [getBean: 
     64                                { what -> 
     65                                        if( what == "authenticationService" ) 
     66                                                return [getLoggedInUser: { return users[0]; } ] 
     67                                        else if( what == "moduleCommunicationService" ) 
     68                                                return null 
     69                                 } 
     70                        ] } 
     71                ] } 
    5472                 
    5573    } 
     
    7088 
    7189                // Search without criteria 
    72                 studySearch.setCriteria( ); 
     90                studySearch.setCriteria(); 
    7391                studySearch.execute(); 
    7492                 
  • trunk/web-app/css/default_style.css

    r1485 r1487  
    196196} 
    197197 
    198 /* DISABLED as this is conflicting with other error styles, 
    199    please rename this in your stuff before enabling again, or 
    200    move it into a separate css file and/or div element */ 
    201 #content .disabledEerror { 
     198#content .errormessage { 
    202199        border: 1px solid #f99; /* #006dba; */ 
    203200        margin-bottom: 10px;