Changeset 860

Timestamp:

Aug 30, 2010, 3:13:30 PM (13 years ago)

Author:

keesvb

Message:

added test for Sample name uniqueness at the validate() stage

Location:

trunk

Files:

• grails-app/conf/BootStrap.groovy (modified) (1 diff)
• test/integration/gscf/SampleTests.groovy (modified) (1 diff)

trunk/grails-app/conf/BootStrap.groovy

@@ -115 +115 @@
 
                         // If in development mode and no studies are present, add example studies
-                        if (Study.count() == 0) { // && grails.util.GrailsUtil.environment == GrailsApplication.ENV_DEVELOPMENT) {
+                        if (Study.count() == 0 && grails.util.GrailsUtil.environment != GrailsApplication.ENV_TEST) {
                                 // check if special file is present in project directory
                                 if ((new File(System.properties['user.dir']+"/.skip-studies").exists())) {

trunk/test/integration/gscf/SampleTests.groovy

@@ -291 +291 @@
         }
 
+        /**
+         * Test whether it's indeed not possible to add two yet-to-be-saved samples with the same name to a yet-to-be-saved study
+         */
+        void testSampleUniqueNameConstraintAtValidate() {
+                def sample = Sample.findByName(testSampleName)
+                assert sample
+
+                def study = sample.parent
+                assert study
+
+                def sample1 = new Sample(
+                    name: testSampleName + "-double",
+                    template: sample.template,
+                    parentEvent: sample.parentEvent
+                )
+
+                def sample2 = new Sample(
+                        name: testSampleName + "-double",
+                    template: sample.template,
+                    parentEvent: sample.parentEvent
+                )
+
+                // Add the sample to the retrieved parent study
+                study.addToSamples(sample1)
+                study.addToSamples(sample2)
+
+                // At this point, the sample should not validate or save, because there is already a sample with that name in the study
+                assert !sample1.validate()
+                assert !sample1.save(flush:true)
+
+                assert !sample2.validate()
+                assert !sample2.save(flush:true)
+        }
+
         void testFindViaSamplingEvent() {
                 // Try to retrieve the sampling event by using the time...