TypeDAL 4.3.5__tar.gz → 4.4.0__tar.gz

typedal-4.4.0/.crush/.gitignore

@@ -0,0 +1 @@
+*

typedal-4.4.0/.crush/logs/crush.log

@@ -0,0 +1,37 @@
+{"time":"2026-01-20T14:19:21.359487106+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/config.(*catwalkSync).Get.func1","file":"github.com/charmbracelet/crush/internal/config/catwalk.go","line":55},"msg":"Fetching providers from Catwalk"}
+{"time":"2026-01-20T14:19:21.518115889+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/config.(*catwalkSync).Get.func1","file":"github.com/charmbracelet/crush/internal/config/catwalk.go","line":63},"msg":"Catwalk providers not modified"}
+{"time":"2026-01-20T14:19:21.52626179+01:00","level":"INFO","msg":"OK   20250424200609_initial.sql (1.52ms)"}
+{"time":"2026-01-20T14:19:21.526651475+01:00","level":"INFO","msg":"OK   20250515105448_add_summary_message_id.sql (330.71µs)"}
+{"time":"2026-01-20T14:19:21.526995425+01:00","level":"INFO","msg":"OK   20250624000000_add_created_at_indexes.sql (325.29µs)"}
+{"time":"2026-01-20T14:19:21.527303588+01:00","level":"INFO","msg":"OK   20250627000000_add_provider_to_messages.sql (293.15µs)"}
+{"time":"2026-01-20T14:19:21.52790648+01:00","level":"INFO","msg":"OK   20250810000000_add_is_summary_message.sql (495.71µs)"}
+{"time":"2026-01-20T14:19:21.528316924+01:00","level":"INFO","msg":"OK   20250812000000_add_todos_to_sessions.sql (389.38µs)"}
+{"time":"2026-01-20T14:19:21.528324939+01:00","level":"INFO","msg":"goose: successfully migrated database to version: 20250812000000"}
+{"time":"2026-01-20T14:19:21.528356959+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/app.(*App).initLSPClients","file":"github.com/charmbracelet/crush/internal/app/lsp.go","line":21},"msg":"LSP clients initialization started in background"}
+{"time":"2026-01-20T14:19:21.52843831+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/app.New.func1","file":"github.com/charmbracelet/crush/internal/app/app.go","line":102},"msg":"Initializing MCP clients"}
+{"time":"2026-01-20T14:21:06.413595152+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T14:21:14.539289381+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T14:21:15.710688424+01:00","level":"WARN","source":{"function":"github.com/charmbracelet/crush/internal/agent/tools.init.func1","file":"github.com/charmbracelet/crush/internal/agent/tools/rg.go","line":18},"msg":"Ripgrep (rg) not found in $PATH. Some grep features might be limited or slower."}
+{"time":"2026-01-20T14:21:17.08089755+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T14:21:17.151106375+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T14:22:05.503761296+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T14:26:05.030425805+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T14:36:26.778665189+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T14:38:51.314473736+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T15:02:22.602405814+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/app.(*App).Shutdown.func1","file":"github.com/charmbracelet/crush/internal/app/app.go","line":403},"msg":"Shutdown took 6.62949ms"}
+{"time":"2026-01-20T15:02:23.719585648+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/config.(*catwalkSync).Get.func1","file":"github.com/charmbracelet/crush/internal/config/catwalk.go","line":55},"msg":"Fetching providers from Catwalk"}
+{"time":"2026-01-20T15:02:23.881789126+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/config.(*catwalkSync).Get.func1","file":"github.com/charmbracelet/crush/internal/config/catwalk.go","line":63},"msg":"Catwalk providers not modified"}
+{"time":"2026-01-20T15:02:23.884044691+01:00","level":"INFO","msg":"goose: no migrations to run. current version: 20250812000000"}
+{"time":"2026-01-20T15:02:23.884082792+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/app.(*App).initLSPClients","file":"github.com/charmbracelet/crush/internal/app/lsp.go","line":21},"msg":"LSP clients initialization started in background"}
+{"time":"2026-01-20T15:02:23.884273487+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/app.New.func1","file":"github.com/charmbracelet/crush/internal/app/app.go","line":102},"msg":"Initializing MCP clients"}
+{"time":"2026-01-20T15:04:13.024457515+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T15:04:14.232072885+01:00","level":"WARN","source":{"function":"github.com/charmbracelet/crush/internal/agent/tools.init.func1","file":"github.com/charmbracelet/crush/internal/agent/tools/rg.go","line":18},"msg":"Ripgrep (rg) not found in $PATH. Some grep features might be limited or slower."}
+{"time":"2026-01-20T15:11:24.612161916+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/app.(*App).Shutdown.func1","file":"github.com/charmbracelet/crush/internal/app/app.go","line":403},"msg":"Shutdown took 9.537141ms"}
+{"time":"2026-01-20T15:11:25.263960575+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/config.(*catwalkSync).Get.func1","file":"github.com/charmbracelet/crush/internal/config/catwalk.go","line":55},"msg":"Fetching providers from Catwalk"}
+{"time":"2026-01-20T15:11:25.431990051+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/config.(*catwalkSync).Get.func1","file":"github.com/charmbracelet/crush/internal/config/catwalk.go","line":63},"msg":"Catwalk providers not modified"}
+{"time":"2026-01-20T15:11:25.434282035+01:00","level":"INFO","msg":"goose: no migrations to run. current version: 20250812000000"}
+{"time":"2026-01-20T15:11:25.434366662+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/app.(*App).initLSPClients","file":"github.com/charmbracelet/crush/internal/app/lsp.go","line":21},"msg":"LSP clients initialization started in background"}
+{"time":"2026-01-20T15:11:25.434579879+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/app.New.func1","file":"github.com/charmbracelet/crush/internal/app/app.go","line":102},"msg":"Initializing MCP clients"}
+{"time":"2026-01-20T15:24:40.456010578+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).generateTitle","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":788},"msg":"generated title with small model"}
+{"time":"2026-01-20T15:24:51.673808433+01:00","level":"WARN","source":{"function":"github.com/charmbracelet/crush/internal/agent/tools.init.func1","file":"github.com/charmbracelet/crush/internal/agent/tools/rg.go","line":18},"msg":"Ripgrep (rg) not found in $PATH. Some grep features might be limited or slower."}
+{"time":"2026-01-20T15:26:03.868420566+01:00","level":"INFO","source":{"function":"github.com/charmbracelet/crush/internal/agent.(*sessionAgent).Cancel","file":"github.com/charmbracelet/crush/internal/agent/agent.go","line":907},"msg":"Request cancellation initiated","session_id":"4c2d1684-8db2-4ffd-83bb-9c08fe850f29"}

{typedal-4.3.5 → typedal-4.4.0}/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 <!--next-version-placeholder-->
 
+## v4.4.0 (2026-01-20)
+
+### Feature
+
+* Add groupby and having methods to QueryBuilder ([`8bce7ec`](https://github.com/trialandsuccess/TypeDAL/commit/8bce7ec137db79ab1631f4504ae891b5bd42c1d8))
+
+## v4.3.6 (2026-01-19)
+
+### Fix
+
+* Ensure typedal caching respects provided 'migrate' enabled config ([`0bcc277`](https://github.com/trialandsuccess/TypeDAL/commit/0bcc2774dcec9231dc8a58438dd70d3ce29650e8))
+
 ## v4.3.5 (2026-01-09)
 
 ### Fix

{typedal-4.3.5 → typedal-4.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: TypeDAL
-Version: 4.3.5
+Version: 4.4.0
 Summary: Typing support for PyDAL
 Project-URL: Documentation, https://typedal.readthedocs.io/
 Project-URL: Issues, https://github.com/trialandsuccess/TypeDAL/issues

{typedal-4.3.5 → typedal-4.4.0}/docs/3_building_queries.md

@@ -22,6 +22,8 @@ A Query Builder can be initialized by calling one of these methods on a TypedTab
 - where
 - select
 - join
+- groupby
+- having
 - cache
 
 e.g. `Person.where(...)` -> `QueryBuilder[Person]`
@@ -155,6 +157,35 @@ Person.join('articles', method='inner')  # will only yield persons that have rel
 
 For more details about relationships and joins, see [4. Relationships](./4_relationships.md).
 
+### groupby & having
+
+Group query results by one or more fields, typically used with aggregate functions like `count()`, `sum()`, `avg()`, etc.
+Use `having` to filter the grouped results based on aggregate conditions.
+
+```python
+# Basic grouping: count articles per author
+Article.select(Article.author, Article.id.count().with_alias("article_count"))
+    .groupby(Article.author)
+    .collect()
+
+# Group by multiple fields
+Sale.select(Sale.product, Sale.region, Sale.amount.sum().with_alias("total"))
+    .groupby(Sale.product, Sale.region)
+    .collect()
+
+# Filter groups with having: only authors with more than 5 articles
+Article.select(Article.author, Article.id.count().with_alias("article_count"))
+    .groupby(Article.author)
+    .having(Article.id.count() > 5)
+    .collect()
+
+# Can be chained in any order
+School.groupby(School.id)
+    .having(Team.id.count() > 0)
+    .select(School.id, Team.id.count())
+    .collect()
+```
+
 ### cache
 
 ```python

{typedal-4.3.5 → typedal-4.4.0}/src/typedal/__about__.py

@@ -5,4 +5,4 @@ This file contains the Version info for this package.
 # SPDX-FileCopyrightText: 2023-present Robin van der Noord <robinvandernoord@gmail.com>
 #
 # SPDX-License-Identifier: MIT
-__version__ = "4.3.5"
+__version__ = "4.4.0"

{typedal-4.3.5 → typedal-4.4.0}/src/typedal/caching.py

@@ -12,6 +12,7 @@ import dill  # nosec
 from pydal.objects import Field, Rows, Set
 
 from .fields import TypedField
+from .helpers import throw
 from .rows import TypedRows
 from .tables import TypedTable
 from .types import CacheStatus, Query
@@ -177,8 +178,12 @@ def clear_cache() -> None:
 
     Immediately commits
     """
+    db: TypeDAL = _TypedalCache._db or throw(
+        RuntimeError("@define or db.define is not called on typedal caching classes yet!")
+    )
+
     _TypedalCache.truncate("RESTART IDENTITY CASCADE")
-    _TypedalCache._db.commit()
+    db.commit()
 
 
 def clear_expired() -> int:

{typedal-4.3.5 → typedal-4.4.0}/src/typedal/cli.py

@@ -392,7 +392,8 @@ def fake_migrations(
 
     previously_migrated = (
         db(
-            db.ewh_implemented_features.name.belongs(to_fake) & (db.ewh_implemented_features.installed == True)  # noqa E712
+            db.ewh_implemented_features.name.belongs(to_fake)
+            & (db.ewh_implemented_features.installed == True)  # noqa E712
         )
         .select(db.ewh_implemented_features.name)
         .column("name")

{typedal-4.3.5 → typedal-4.4.0}/src/typedal/core.py

@@ -4,6 +4,7 @@ Core functionality of TypeDAL.
 
 from __future__ import annotations
 
+import datetime as dt
 import sys
 import typing as t
 import warnings
@@ -246,7 +247,7 @@ class TypeDAL(pydal.DAL):
         Try to define a model with migrate or fall back to fake migrate.
         """
         try:
-            return self.define(model, migrate=True)
+            return self.define(model, migrate=self._migrate)
         except Exception as e:
             # clean up:
             self.rollback()
@@ -257,7 +258,7 @@ class TypeDAL(pydal.DAL):
                 warnings.warn(f"{model} could not be migrated, try faking", source=e, category=RuntimeWarning)
 
             # try again:
-            return self.define(model, migrate=True, fake_migrate=True, redefine=True)
+            return self.define(model, migrate=self._migrate, fake_migrate=self._migrate, redefine=True)
 
     default_kwargs: t.ClassVar[AnyDict] = {
         # fields are 'required' (notnull) by default:

{typedal-4.3.5 → typedal-4.4.0}/src/typedal/query_builder.py

@@ -70,8 +70,7 @@ class QueryBuilder(t.Generic[T_MetaInstance]):
         """
         self.model = model
         table = self._ensure_table_defined()
-
-        default_query = table.id > 0
+        default_query: Query = table.id > 0
         self.query = add_query or default_query
         self.select_args = select_args or []
         self.select_kwargs = select_kwargs or {}
@@ -111,7 +110,7 @@ class QueryBuilder(t.Generic[T_MetaInstance]):
         Querybuilder is truthy if it has t.Any conditions.
         """
         table = self._ensure_table_defined()
-        default_query = table.id > 0
+        default_query: Query = table.id > 0
         return any(
             [
                 self.query != default_query,
@@ -183,6 +182,31 @@ class QueryBuilder(t.Generic[T_MetaInstance]):
         """
         return self.select(orderby=fields)
 
+    def groupby(self, *fields: t.Any) -> "QueryBuilder[T_MetaInstance]":
+        """
+        Group the query results by specified fields.
+
+        Args:
+            fields: Field(s) to group by (e.g., Table.column)
+
+        Returns:
+            QueryBuilder: A new QueryBuilder instance with grouping applied.
+        """
+        groupby_value = fields[0] if len(fields) == 1 else fields
+        return self.select(groupby=groupby_value)
+
+    def having(self, condition: t.Any) -> "QueryBuilder[T_MetaInstance]":
+        """
+        Filter grouped query results based on aggregate conditions.
+
+        Args:
+            condition: Query condition for filtering groups (e.g., Team.id.count() > 0)
+
+        Returns:
+            QueryBuilder: A new QueryBuilder instance with having condition applied.
+        """
+        return self.select(having=condition)
+
     def where(
         self,
         *queries_or_lambdas: Query | t.Callable[[t.Type[T_MetaInstance]], Query] | dict[str, t.Any],
@@ -524,7 +548,7 @@ class QueryBuilder(t.Generic[T_MetaInstance]):
         Raw version of .collect which only executes the SQL, without performing t.Any magic afterwards.
         """
         db = self._get_db()
-        metadata = self.metadata.copy()
+        metadata: Metadata = self.metadata.copy()
 
         query, select_args, select_kwargs = self._before_query(metadata, add_id=add_id)
 
@@ -552,7 +576,7 @@ class QueryBuilder(t.Generic[T_MetaInstance]):
         for fn_before in db._before_collect:
             fn_before(self)
 
-        metadata = self.metadata.copy()
+        metadata: Metadata = self.metadata.copy()
 
         if metadata.get("cache", {}).get("enabled") and (result := self._collect_cached(metadata)):
             return result

{typedal-4.3.5 → typedal-4.4.0}/src/typedal/tables.py

@@ -326,6 +326,18 @@ class TableMeta(type):
         """
         return QueryBuilder(self).orderby(*fields)
 
+    def groupby(self: t.Type[T_MetaInstance], *fields: t.Any) -> "QueryBuilder[T_MetaInstance]":
+        """
+        See QueryBuilder.groupby!
+        """
+        return QueryBuilder(self).groupby(*fields)
+
+    def having(self: t.Type[T_MetaInstance], condition: t.Any) -> "QueryBuilder[T_MetaInstance]":
+        """
+        See QueryBuilder.having!
+        """
+        return QueryBuilder(self).having(condition)
+
     def cache(self: t.Type[T_MetaInstance], *deps: t.Any, **kwargs: t.Any) -> "QueryBuilder[T_MetaInstance]":
         """
         See QueryBuilder.cache!

{typedal-4.3.5 → typedal-4.4.0}/src/typedal/types.py

@@ -220,6 +220,8 @@ class SelectKwargs(t.TypedDict, total=False):
     join: t.Optional[list[Expression]]
     left: t.Optional[list[Expression]]
     orderby: "OrderBy | t.Iterable[OrderBy] | None"
+    groupby: "GroupBy | t.Iterable[GroupBy] | None"
+    having: "Having | None"
     limitby: t.Optional[tuple[int, int]]
     distinct: bool | Field | Expression
     orderby_on_limitby: bool
@@ -323,5 +325,7 @@ CacheModel = t.Callable[[str, CacheFn, int], Rows]
 CacheTuple = tuple[CacheModel, int]
 
 OrderBy: t.TypeAlias = str | Expression
+GroupBy: t.TypeAlias = Field | Expression
+Having: t.TypeAlias = Query | Expression
 
 T_annotation = t.Type[t.Any] | types.UnionType

{typedal-4.3.5 → typedal-4.4.0}/tests/test_config.py

@@ -1,6 +1,7 @@
 import datetime as dt
 import os
 import shutil
+import sqlite3
 import tempfile
 import uuid
 from pathlib import Path
@@ -331,3 +332,20 @@ def test_uuid_fields_sqlite(at_temp_dir):
         UUIDTable.insert(gid="not-a-uuid")
 
     assert '"gid" uuid NOT NULL' in UUIDTable._sql()
+
+
+def test_cache_migrate_disabled():
+    try:
+        os.environ["TYPEDAL_MIGRATE"] = "0"
+
+        db = TypeDAL("sqlite:memory")
+
+        assert db._migrate == False
+        with pytest.raises(sqlite3.OperationalError):
+            # should raise NotFound because migrate is disabled:
+            assert db.executesql("""
+            select * from typedal_cache;
+            """)
+
+    finally:
+        del os.environ["TYPEDAL_MIGRATE"]

{typedal-4.3.5 → typedal-4.4.0}/tests/test_query_builder.py

@@ -45,25 +45,6 @@ def test_query_type():
     assert isinstance(TestQueryTable.number != 3, Query)
 
 
-"""
-SELECT "test_query_table"."id"
-     , "test_query_table"."number"
-     , "relations_8106139955393"."id"
-     , "relations_8106139955393"."name"
-     , "relations_8106139955393"."value"
-     , "relations_8106139955393"."querytable"
-    FROM "test_query_table"
-             LEFT JOIN "test_relationship" AS "relations_8106139955393"
-                       ON ("relations_8106139955393"."querytable" = "test_query_table"."id")
-    WHERE ("test_query_table"."id" IN (SELECT "test_query_table"."id"
-                                           FROM "test_query_table"
-                                           WHERE ("test_query_table"."id" > 0)
-                                           ORDER BY "test_query_table"."id"
-                                           LIMIT 3 OFFSET 0))
-    ORDER BY "test_query_table"."number" DESC;
-"""
-
-
 def _setup_data():
     TestQueryTable.truncate()
     first = TestQueryTable.insert(number=0)
@@ -562,3 +543,131 @@ def test_before_after_collect():
     db._after_collect.append(print_duration)
 
     TestQueryTable.all()
+
+
+def test_groupby_basic():
+    """Test basic groupby with count aggregation."""
+    _setup_data()
+
+    result = TestRelationship.select(
+        TestRelationship.querytable.with_alias("query_table"),
+        TestRelationship.querytable.count().with_alias("count"),
+    ).groupby(TestRelationship.querytable).execute()
+
+    assert len(result) == 2
+    for row in result:
+        assert row["count"] == 4
+
+
+def test_groupby_multiple_fields():
+    """Test grouping by multiple fields."""
+    _setup_data()
+
+    result = TestRelationship.select(
+        TestRelationship.querytable,
+        TestRelationship.value,
+        TestRelationship.id.count().with_alias("count"),
+    ).groupby(TestRelationship.querytable, TestRelationship.value).execute()
+
+    # Should group by combination of querytable and value
+    assert len(result) > 0
+
+
+def test_groupby_with_having():
+    """Test groupby with having to filter groups."""
+    _setup_data()
+
+    result = TestRelationship.select(
+        TestRelationship.querytable.with_alias("query_table"),
+        TestRelationship.querytable.count().with_alias("count"),
+    ).groupby(TestRelationship.querytable).having(TestRelationship.querytable.count() > 3).execute()
+
+    # Only groups with count > 3
+    assert len(result) == 2
+    for row in result:
+        assert row["count"] > 3
+
+
+def test_having_filters_aggregates():
+    """Test that having properly filters based on aggregate conditions."""
+    _setup_data()
+
+    # Get all groups
+    all_groups = TestRelationship.select(
+        TestRelationship.querytable,
+        TestRelationship.querytable.count().with_alias("count"),
+    ).groupby(TestRelationship.querytable).execute()
+
+    # Filter with having
+    filtered = TestRelationship.select(
+        TestRelationship.querytable,
+        TestRelationship.querytable.count().with_alias("count"),
+    ).groupby(TestRelationship.querytable).having(TestRelationship.querytable.count() > 10).execute()
+
+    # Should have fewer results (or zero if no groups have count > 10)
+    assert len(filtered) <= len(all_groups)
+
+
+def test_groupby_to_sql():
+    """Verify SQL generation includes GROUP BY."""
+    sql = TestRelationship.select(
+        TestRelationship.querytable, TestRelationship.querytable.count()
+    ).groupby(TestRelationship.querytable).to_sql()
+
+    assert "GROUP BY" in sql
+
+
+def test_having_to_sql():
+    """Verify SQL generation includes HAVING."""
+    sql = (
+        TestRelationship.select(TestRelationship.querytable, TestRelationship.querytable.count())
+        .groupby(TestRelationship.querytable)
+        .having(TestRelationship.querytable.count() > 0)
+        .to_sql()
+    )
+
+    assert "GROUP BY" in sql
+    assert "HAVING" in sql
+
+
+def test_groupby_chaining():
+    """Test that multiple groupby calls work (last one should win)."""
+    _setup_data()
+
+    # First groupby by querytable
+    builder1 = TestRelationship.select(
+        TestRelationship.querytable, TestRelationship.querytable.count().with_alias("count")
+    ).groupby(TestRelationship.querytable)
+
+    # Then groupby by value (should override)
+    builder2 = builder1.groupby(TestRelationship.value)
+
+    sql = builder2.to_sql()
+    # Should only have the second groupby
+    assert "GROUP BY" in sql
+
+
+def test_groupby_having_on_table_class():
+    """Test calling .groupby() and .having() directly on table class in different orders."""
+    _setup_data()
+
+    builder1 = (
+        TestRelationship.groupby(TestRelationship.querytable)
+        .having(TestRelationship.querytable.count() > 0)
+        .select(TestRelationship.querytable, TestRelationship.querytable.count())
+    )
+
+    sql1 = builder1.to_sql()
+
+    builder2 = (
+        TestRelationship.having(TestRelationship.querytable.count() > 0)
+        .groupby(TestRelationship.querytable)
+        .select(TestRelationship.querytable, TestRelationship.querytable.count())
+    )
+    sql2 = builder2.to_sql()
+
+    assert sql1 == sql2
+    assert "GROUP BY" in sql1
+    assert "HAVING" in sql1
+
+    assert builder1.execute() == builder2.execute()