Skip to content

Commit

Permalink
remove support for coffeescript
Browse files Browse the repository at this point in the history
  • Loading branch information
klieber committed Sep 14, 2016
1 parent f488466 commit 7a5a14e
Show file tree
Hide file tree
Showing 11 changed files with 6 additions and 494 deletions.
49 changes: 0 additions & 49 deletions src/main/java/com/github/searls/jasmine/coffee/CoffeeScript.java

This file was deleted.

11 changes: 0 additions & 11 deletions src/main/java/com/github/searls/jasmine/coffee/DetectsCoffee.java

This file was deleted.

This file was deleted.

Expand Up @@ -44,7 +44,5 @@ public interface JasmineConfiguration {

int getAutoRefreshInterval();

boolean isCoffeeScriptCompilationEnabled();

ClassLoader getProjectClassLoader();
}
Expand Up @@ -276,15 +276,13 @@ public abstract class AbstractJasmineMojo extends AbstractMojo implements Jasmin
* <include>jasmine-jquery.js</include>
* <include>spec-helper.js</include>
* <include>**/*.js</include>
* <include>**/*.coffee</include>
* </specIncludes>
* </pre>
* <br>
* <p>Default <code>specIncludes</code>:</p>
* <pre>
* &lt;specIncludes&gt;
* &lt;include&gt;&#42;&#42;/&#42;.js&lt;/include&gt;
* &lt;include&gt;&#42;&#42;/&#42;.coffee&lt;/include&gt;
* &lt;/specIncludes&gt;
* </pre>
*
Expand Down Expand Up @@ -351,16 +349,6 @@ public abstract class AbstractJasmineMojo extends AbstractMojo implements Jasmin
@Parameter(property = "jasmine.autoRefreshInterval", defaultValue = "0")
protected int autoRefreshInterval;

/**
* <p>Control the Coffee Script compilation. e.g. When using RequireJS the compilation
* happens within the Coffee Script AMD loader plugin; we therefore need to disable the
* compilation here.</p>
*
* @since 1.3.1.4
*/
@Parameter(property = "coffeeScriptCompilationEnabled", defaultValue = "true")
protected boolean coffeeScriptCompilationEnabled;

/**
* <p>Type of {@link org.eclipse.jetty.server.Connector} to use on the jetty server.</p>
* <br>
Expand Down Expand Up @@ -480,11 +468,6 @@ public int getAutoRefreshInterval() {
return this.autoRefreshInterval;
}

@Override
public boolean isCoffeeScriptCompilationEnabled() {
return this.coffeeScriptCompilationEnabled;
}

public MavenProject getMavenProject() {
return this.mavenProject;
}
Expand Down
@@ -1,8 +1,5 @@
package com.github.searls.jasmine.server;

import com.github.searls.jasmine.coffee.DetectsCoffee;
import com.github.searls.jasmine.coffee.HandlesRequestsForCoffee;
import com.github.searls.jasmine.config.JasmineConfiguration;
import com.github.searls.jasmine.runner.CreatesRunner;
import org.eclipse.jetty.server.Request;
import org.eclipse.jetty.server.handler.ResourceHandler;
Expand All @@ -15,20 +12,10 @@

public class JasmineResourceHandler extends ResourceHandler {

private final DetectsCoffee detectsCoffee;
private final HandlesRequestsForCoffee handlesRequestsForCoffee;
private final CreatesRunner createsRunner;

public JasmineResourceHandler(CreatesRunner createsRunner, JasmineConfiguration configuration) {
this(createsRunner, new HandlesRequestsForCoffee(configuration), new DetectsCoffee());
}

public JasmineResourceHandler(CreatesRunner createsRunner,
HandlesRequestsForCoffee handlesRequestsForCoffee,
DetectsCoffee detectsCoffee) {
this.detectsCoffee = detectsCoffee;
public JasmineResourceHandler(CreatesRunner createsRunner) {
this.createsRunner = createsRunner;
this.handlesRequestsForCoffee = handlesRequestsForCoffee;
setAliases(true);
}

Expand All @@ -37,22 +24,13 @@ public void handle(String target, Request baseRequest, HttpServletRequest reques
this.createSpecRunnerIfNecessary(target);
Resource resource = this.getResource(baseRequest);
response.addDateHeader("EXPIRES", 0L);
if (this.detectsCoffee.detect(target) && this.weCanHandleIt(baseRequest, resource)) {
this.handlesRequestsForCoffee.handle(baseRequest, response, resource);
} else {
//Not testable. Who knew test-driving an LSP violation would be this hard. Sigh. :-(
super.handle(target, baseRequest, baseRequest, response);
}
//Not testable. Who knew test-driving an LSP violation would be this hard. Sigh. :-(
super.handle(target, baseRequest, baseRequest, response);
}

private void createSpecRunnerIfNecessary(String target) throws IOException {
if ("/".equals(target)) {
this.createsRunner.create();
}
}

private boolean weCanHandleIt(Request baseRequest, Resource resource) {
return !baseRequest.isHandled() && resource != null && resource.exists();
}

}
8 changes: 0 additions & 8 deletions src/main/resources/vendor/js/coffee-script.js

This file was deleted.

This file was deleted.

This file was deleted.

0 comments on commit 7a5a14e

Please sign in to comment.