Skip to content

Commit e266458

Browse files
committed
updated tests to ignore internal schemas and removed unused sql queries
1 parent 4c88244 commit e266458

File tree

10 files changed

+7
-37
lines changed

10 files changed

+7
-37
lines changed

cmd/inspect.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -224,11 +224,11 @@ func printReportSummary(outDir string) error {
224224
table := "RULE|STATUS|MATCHES\n|-|-|-|\n"
225225

226226
// find matching rule
227-
227+
var status string
228228
for i := range cfg.Rules {
229229
r := &cfg.Rules[i]
230230
name := r.Name
231-
status := "--"
231+
232232
row := db.QueryRow(r.Query)
233233
var match sql.NullString
234234

internal/inspect/db_stats/db_stats_test.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@ import (
77
"github.com/jackc/pgconn"
88
"github.com/spf13/afero"
99
"github.com/stretchr/testify/assert"
10-
"github.com/supabase/cli/internal/db/reset"
11-
"github.com/supabase/cli/internal/utils"
1210
"github.com/supabase/cli/pkg/pgtest"
1311
)
1412

@@ -27,7 +25,7 @@ func TestDBStatsCommand(t *testing.T) {
2725
// Setup mock postgres
2826
conn := pgtest.NewConn()
2927
defer conn.Close(t)
30-
conn.Query(DBStatsQuery, reset.LikeEscapeSchema(utils.InternalSchemas)).
28+
conn.Query(DBStatsQuery).
3129
Reply("SELECT 1", Result{
3230
Database_size: "8GB",
3331
Total_index_size: "8GB",

internal/inspect/index_stats/index_stats_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ func TestIndexStatsCommand(t *testing.T) {
2727
defer conn.Close(t)
2828

2929
// Mock index stats
30-
conn.Query(IndexStatsQuery, reset.LikeEscapeSchema(utils.PgSchemas)).
30+
conn.Query(IndexStatsQuery, reset.LikeEscapeSchema(utils.InternalSchemas)).
3131
Reply("SELECT 1", Result{
3232
Name: "public.test_idx",
3333
Size: "1GB",

internal/inspect/seq_scans/seq_scans.sql

Lines changed: 0 additions & 6 deletions
This file was deleted.

internal/inspect/table_index_sizes/table_index_sizes.sql

Lines changed: 0 additions & 8 deletions
This file was deleted.

internal/inspect/table_record_counts/table_record_counts.sql

Lines changed: 0 additions & 6 deletions
This file was deleted.

internal/inspect/table_sizes/table_sizes.sql

Lines changed: 0 additions & 8 deletions
This file was deleted.

internal/inspect/table_stats/table_stats_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ func TestTableStatsCommand(t *testing.T) {
2727
defer conn.Close(t)
2828

2929
// Mock table sizes and index sizes
30-
conn.Query(TableStatsQuery, reset.LikeEscapeSchema(utils.PgSchemas)).
30+
conn.Query(TableStatsQuery, reset.LikeEscapeSchema(utils.InternalSchemas)).
3131
Reply("SELECT 1", Result{
3232
Name: "public.test_table",
3333
Table_size: "3GB",

internal/inspect/total_table_sizes/total_table_sizes.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ func Run(ctx context.Context, config pgconn.Config, fsys afero.Fs, options ...fu
2929
return err
3030
}
3131
defer conn.Close(context.Background())
32-
rows, err := conn.Query(ctx, TotalTableSizesQuery, reset.LikeEscapeSchema(utils.PgSchemas))
32+
rows, err := conn.Query(ctx, TotalTableSizesQuery, reset.LikeEscapeSchema(utils.InternalSchemas))
3333
if err != nil {
3434
return errors.Errorf("failed to query rows: %w", err)
3535
}

internal/inspect/total_table_sizes/total_table_sizes_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ func TestTotalTableSizesCommand(t *testing.T) {
2727
// Setup mock postgres
2828
conn := pgtest.NewConn()
2929
defer conn.Close(t)
30-
conn.Query(TotalTableSizesQuery, reset.LikeEscapeSchema(utils.PgSchemas)).
30+
conn.Query(TotalTableSizesQuery, reset.LikeEscapeSchema(utils.InternalSchemas)).
3131
Reply("SELECT 1", Result{
3232
Name: "public.test_table",
3333
Size: "3GB",

0 commit comments

Comments
 (0)