Deleting javascript resource extension throws NullPointerException

[sravanr]

I see a NullPointerException when am trying to delete a javascript resource extension

resourceMgr.deleteServices("simpleJSResourceModule");

public void setUp() throws Exception {
         client = DatabaseClientFactory.newClient("localhost", restPort, "rest-admin", "x", Authentication.DIGEST);
         resourceMgr = client.newServerConfigManager().newResourceExtensionsManager();
         ExtensionMetadata resextMetadata = new ExtensionMetadata();
         resextMetadata.setTitle("BasicJSTest");
         resextMetadata.setDescription("Testing resource extension for java script");
         System.out.println(resextMetadata.getScriptLanguage());
         resextMetadata.setScriptLanguage(ScriptLanguage.JAVASCRIPT);
         System.out.println(resextMetadata.getScriptLanguage());
         resextMetadata.setVersion("1.0");
         MethodParameters getParams = new MethodParameters(MethodType.GET);
         getParams.add("my-uri", "xs:string?");
         FileInputStream myStream = new FileInputStream("src/test/java/com/marklogic/javaclient/data/JSResource.js");
         InputStreamHandle handle = new InputStreamHandle(myStream);
         handle.set (myStream);
         resourceMgr.writeServices("simpleJSResourceModule", handle, resextMetadata,getParams);


    } 
@After
    public void tearDown() throws Exception {
        client.release();
        resourceMgr.deleteServices("simpleJSResourceModule");
    }


here is the stack trace

java.lang.NullPointerException
    at com.marklogic.client.impl.JerseyServices.deleteValue(JerseyServices.java:2927)
    at com.marklogic.client.impl.ResourceExtensionsImpl.deleteServices(ResourceExtensionsImpl.java:217)
    at com.marklogic.javaclient.TestJSResourceExtensions.tearDown(TestJSResourceExtensions.java:135)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
    at java.lang.reflect.Method.invoke(Unknown Source)
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
    at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
    at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:33)
    at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
    at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
    at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
    at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
    at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
    at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
    at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
    at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
    at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:50)
    at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:467)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:683)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:390)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:197)

[sammefford]

You cannot use your resourceMgr after you've called client.release(). But we should throw a helpful error message not NPE. So I've made a change which throws a helpful error message, and I've added negative unit tests to make sure it's thrown.

[sravanr]

I see the expected IllegalStateException now.