fix(query): predicate bug

crates/turborepo-lib/src/query/mod.rs

@@ -385,7 +385,8 @@ impl PackagePredicate {
         let less_than = self
             .less_than
             .as_ref()
-            .map(|pair| Self::check_greater_than(pkg, &pair.field, &pair.value));
+            .map(|pair| Self::check_less_than(pkg, &pair.field, &pair.value));
+
         let not = self.not.as_ref().map(|predicate| !predicate.check(pkg));
         let has = self
             .has

crates/turborepo-lib/src/query/package.rs

@@ -1,4 +1,4 @@
-use std::{collections::HashMap, sync::Arc};
+use std::{collections::HashMap, fmt, sync::Arc};
 
 use async_graphql::Object;
 use itertools::Itertools;
@@ -16,6 +16,12 @@ pub struct Package {
     name: PackageName,
 }
 
+impl fmt::Debug for Package {
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        f.debug_struct("Package").field("name", &self.name).finish()
+    }
+}
+
 impl Package {
     pub fn new(run: Arc<Run>, name: PackageName) -> Result<Self, Error> {
         run.pkg_dep_graph()

crates/turborepo/tests/query.rs

@@ -7,6 +7,8 @@ fn test_query() -> Result<(), anyhow::Error> {
         "npm@10.5.0",
         "query",
         "get package that doesn't exist" => "query { package(name: \"doesnotexist\") { path } }",
+        "get packages with less than 1 dependents" => "query { packages(filter: {lessThan: {field: DIRECT_DEPENDENT_COUNT, value: 1}}) { items { name directDependents { length } } } }",
+        "get packages with more than 0 dependents" => "query { packages(filter: {greaterThan: {field: DIRECT_DEPENDENT_COUNT, value: 0}}) { items { name directDependents { length } } } }",
     );
 
     Ok(())

crates/turborepo/tests/snapshots/query__basic_monorepo_get_packages_with_less_than_1_dependents_(npm@10.5.0).snap

@@ -0,0 +1,30 @@
+---
+source: crates/turborepo/tests/query.rs
+expression: query_output
+---
+{
+  "data": {
+    "packages": {
+      "items": [
+        {
+          "name": "//",
+          "directDependents": {
+            "length": 0
+          }
+        },
+        {
+          "name": "another",
+          "directDependents": {
+            "length": 0
+          }
+        },
+        {
+          "name": "my-app",
+          "directDependents": {
+            "length": 0
+          }
+        }
+      ]
+    }
+  }
+}

crates/turborepo/tests/snapshots/query__basic_monorepo_get_packages_with_less_than_2_dependents_(npm@10.5.0).snap

@@ -0,0 +1,36 @@
+---
+source: crates/turborepo/tests/query.rs
+expression: query_output
+---
+{
+  "data": {
+    "packages": {
+      "items": [
+        {
+          "name": "//",
+          "directDependents": {
+            "length": 0
+          }
+        },
+        {
+          "name": "another",
+          "directDependents": {
+            "length": 0
+          }
+        },
+        {
+          "name": "my-app",
+          "directDependents": {
+            "length": 0
+          }
+        },
+        {
+          "name": "util",
+          "directDependents": {
+            "length": 1
+          }
+        }
+      ]
+    }
+  }
+}

crates/turborepo/tests/snapshots/query__basic_monorepo_get_packages_with_more_than_0_dependents_(npm@10.5.0).snap

@@ -0,0 +1,18 @@
+---
+source: crates/turborepo/tests/query.rs
+expression: query_output
+---
+{
+  "data": {
+    "packages": {
+      "items": [
+        {
+          "name": "util",
+          "directDependents": {
+            "length": 1
+          }
+        }
+      ]
+    }
+  }
+}