Changeset 1487


Ignore:
Timestamp:
Feb 3, 2011, 10:51:16 AM (6 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 edited

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;
Note: See TracChangeset for help on using the changeset viewer.