JS: address non-semantic review feedback

Author: esbena

javascript/ql/src/experimental/poi/PoI.qll

@@ -36,12 +36,12 @@
  *   override predicate shown(DataFlow::Node n) { n.getFile().getBaseName() = "server-core.js" }
  * }
  *
- * class RouteHandlerPoI extends DefaultEnabledPoI {
+ * class RouteHandlerPoI extends ActivePoI {
  *   RouteHandlerPoI() { this = "RouteHandlerPoI" }
  *   override predicate is(DataFlow::Node l0) { l0 instanceof Express::RouteHandler }
  * }
  *
- * class RouteSetupAndRouteHandlerPoI extends DefaultEnabledPoI {
+ * class RouteSetupAndRouteHandlerPoI extends ActivePoI {
  *   RouteSetupAndRouteHandlerPoI() { this = "RouteSetupAndRouteHandlerPoI" }
  *
  *   override predicate is(DataFlow::Node l0, DataFlow::Node l1, string t1) {
@@ -62,11 +62,11 @@ private import semmle.javascript.RestrictedLocations
  * Provides often used points of interest.
  *
  * Note that these points of interest should not extend
- * `DefaultEnabledPoI`, and that they can be enabled by default on
+ * `ActivePoI`, and that they can be enabled on
  * demand like this:
  *
  * ```
- * class MyPoI extends ServerRelatedPoI, DefaultEnabledPoI {}
+ * class MyPoI extends ServerRelatedPoI, ActivePoI {}
  * ```
  */
 private module StandardPoIs {
@@ -172,11 +172,11 @@ private module StandardPoIs {
   }
 
   /**
-   * Provides groups of often used point of interest.
+   * Provides groups of often used points of interest.
    */
   module StandardPoIGroups {
     /**
-     * A server-related points of interest.
+     * A server-related point of interest.
      */
     class ServerRelatedPoI extends PoI {
       ServerRelatedPoI() {
@@ -187,7 +187,7 @@ private module StandardPoIs {
     }
 
     /**
-     * A configuration-related points of interest.
+     * A configuration-related point of interest.
      */
     class DataFlowConfigurationPoI extends PoI {
       DataFlowConfigurationPoI() {
@@ -207,13 +207,13 @@ import StandardPoIs
  * enabled in the absence of an explicit
  * `PoIConfiguration::enabled/1`.
  */
-abstract class DefaultEnabledPoI extends PoI {
+abstract class ActivePoI extends PoI {
   bindingset[this]
-  DefaultEnabledPoI() { any() }
+  ActivePoI() { any() }
 }
 
 private module PoIConfigDefaults {
-  predicate enabled(PoI poi) { poi instanceof DefaultEnabledPoI }
+  predicate enabled(PoI poi) { poi instanceof ActivePoI }
 
   predicate shown(Node n) { not classify(n.getFile(), _) }
 }

javascript/ql/test/experimental/PoI/ServerPoIs.ql

@@ -5,6 +5,6 @@
 import javascript
 import experimental.poi.PoI
 
-class MyServerRelatedPoI extends ServerRelatedPoI, DefaultEnabledPoI {}
+class MyServerRelatedPoI extends ServerRelatedPoI, ActivePoI {}
 
 query predicate problems = alertQuery/6;

javascript/ql/test/experimental/PoI/TestCustomPoIs.ql

@@ -6,21 +6,21 @@ import javascript
 import experimental.poi.PoI
 import DataFlow
 
-class RouteHandlerPoI extends DefaultEnabledPoI {
+class RouteHandlerPoI extends ActivePoI {
   RouteHandlerPoI() { this = "RouteHandlerPoI" }
 
   override predicate is(Node l0) { l0 instanceof Express::RouteHandler }
 }
 
-class RouteHandlerAndSetupPoI extends DefaultEnabledPoI {
+class RouteHandlerAndSetupPoI extends ActivePoI {
   RouteHandlerAndSetupPoI() { this = "RouteHandlerAndSetupPoI" }
 
   override predicate is(Node l0, Node l1, string t1) {
     l1.asExpr().(Express::RouteSetup).getARouteHandler() = l0 and t1 = "setup"
   }
 }
 
-class RouteSetupAndRouterAndRouteHandlerPoI extends DefaultEnabledPoI {
+class RouteSetupAndRouterAndRouteHandlerPoI extends ActivePoI {
   RouteSetupAndRouterAndRouteHandlerPoI() { this = "RouteSetupAndRouterAndRouteHandlerPoI" }
 
   override predicate is(Node l0, Node l1, string t1, Node l2, string t2) {
@@ -31,4 +31,4 @@ class RouteSetupAndRouterAndRouteHandlerPoI extends DefaultEnabledPoI {
   }
 }
 
-query predicate problems = alertQuery/6;
+query predicate problems = alertQuery/6;