Grails

Unable to load file as ClasspathResource in test/unit directory

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.1-RC1
  • Fix Version/s: 1.1-RC2
  • Component/s: Testing
  • Labels:
    None
  • Environment:
    windows

Description

This worked in 1.0.4, but does not seem to work in 1.1-RC1. I'd like to load a file from the test/unit or test/integration directory via ClasspathResource. If there's a better place to put test files, let me know.

Here are the steps to re-create:
1. grails create-app fileloader
2. grails create-service FileLoader
3. add the following code to FileLoaderServiceTests.testSomething()

Resource resource = new ClassPathResource('file.txt')
File f = resource.getFile()
Reader r = f.newReader()
r.eachLine

{ assertEquals 'hello', it }

4. create file file.txt in test/unit directory. contains a single line of text 'hello'
5. grails test-app fails w/ the following:
class path resource [file.txt] cannot be resolved to URL because it does not exist

java.io.FileNotFoundException: class path resource [file.txt] cannot be resolved to URL because it does not exist

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Peter Ledbrook added a comment -

I'd prefer a dedicated directory for resources, much like Maven has. Something like test/resources.

Show
Peter Ledbrook added a comment - I'd prefer a dedicated directory for resources, much like Maven has. Something like test/resources .
Hide
Permalink
David Reines added a comment -

I like the idea of something like test/resources.

Show
David Reines added a comment - I like the idea of something like test/resources.
Hide
Permalink
Peter Ledbrook added a comment -

The workaround is to specify the class loader:

Resource resource = new ClassPathResources("file.txt", getClass().classLoader)
...

To fix this, I would have to update the context class loader, but I need to think on whether that's the right thing to do or not.

We won't add a "test/resources" directory in this version, so I'll raise an issue for it to go into the next one.

Show
Peter Ledbrook added a comment - The workaround is to specify the class loader: Resource resource = new ClassPathResources( "file.txt" , getClass().classLoader) ... To fix this, I would have to update the context class loader, but I need to think on whether that's the right thing to do or not. We won't add a "test/resources" directory in this version, so I'll raise an issue for it to go into the next one.
Hide
Permalink
Peter Ledbrook added a comment -

The source folders are now on the context class loader's path when running the relevant tests.

Show
Peter Ledbrook added a comment - The source folders are now on the context class loader's path when running the relevant tests.
Hide
Permalink
Graeme Rocher added a comment -

Bulk closing bunch of resolved issues

Show
Graeme Rocher added a comment - Bulk closing bunch of resolved issues

People

  • Assignee:
    Peter Ledbrook
    Reporter:
    David Reines
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: