Skip to content

Commit

Permalink
Update TODO comments (#3262)
Browse files Browse the repository at this point in the history
  • Loading branch information
noisersup authored Aug 29, 2023
1 parent e0b4073 commit eccf40d
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 12 deletions.
4 changes: 2 additions & 2 deletions integration/commands_administration_compat_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ func TestCommandsAdministrationCompatCollStatsWithScale(t *testing.T) {
tt.Helper()

tt.Parallel()
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")

var targetRes bson.D
targetCommand := bson.D{{"collStats", targetCollection.Name()}, {"scale", tc.scale}}
Expand Down Expand Up @@ -124,7 +124,7 @@ func TestCommandsAdministrationCompatDBStatsWithScale(t *testing.T) {

tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")

var targetRes bson.D
targetCommand := bson.D{{"dbStats", int32(1)}, {"scale", tc.scale}}
Expand Down
16 changes: 8 additions & 8 deletions integration/commands_administration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ func TestCommandsAdministrationCreateDropListDatabases(t *testing.T) {
func TestCommandsAdministrationListDatabases(tt *testing.T) {
tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3260")
ctx, collection := setup.Setup(t, shareddata.DocumentsStrings)

db := collection.Database()
Expand Down Expand Up @@ -679,7 +679,7 @@ func TestCommandsAdministrationCollStatsEmpty(tt *testing.T) {
tt.Parallel()
ctx, collection := setup.Setup(tt)

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")

var actual bson.D
command := bson.D{{"collStats", collection.Name()}}
Expand All @@ -702,7 +702,7 @@ func TestCommandsAdministrationCollStatsEmpty(tt *testing.T) {
func TestCommandsAdministrationCollStats(tt *testing.T) {
tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")
ctx, collection := setup.Setup(t, shareddata.DocumentsStrings)

var actual bson.D
Expand Down Expand Up @@ -735,7 +735,7 @@ func TestCommandsAdministrationCollStats(tt *testing.T) {
func TestCommandsAdministrationCollStatsWithScale(tt *testing.T) {
tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")

ctx, collection := setup.Setup(t, shareddata.DocumentsStrings)

Expand Down Expand Up @@ -855,7 +855,7 @@ func TestCommandsAdministrationDataSizeErrors(tt *testing.T) {
func TestCommandsAdministrationDBStats(tt *testing.T) {
tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")
ctx, collection := setup.Setup(t, shareddata.DocumentsStrings)

var actual bson.D
Expand Down Expand Up @@ -887,7 +887,7 @@ func TestCommandsAdministrationDBStats(tt *testing.T) {
func TestCommandsAdministrationDBStatsEmpty(tt *testing.T) {
tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")
ctx, collection := setup.Setup(t)

var actual bson.D
Expand All @@ -912,7 +912,7 @@ func TestCommandsAdministrationDBStatsEmpty(tt *testing.T) {
func TestCommandsAdministrationDBStatsWithScale(tt *testing.T) {
tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")
ctx, collection := setup.Setup(t, shareddata.DocumentsStrings)

var actual bson.D
Expand All @@ -937,7 +937,7 @@ func TestCommandsAdministrationDBStatsWithScale(tt *testing.T) {
func TestCommandsAdministrationDBStatsEmptyWithScale(tt *testing.T) {
tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3259")
ctx, collection := setup.Setup(t)

var actual bson.D
Expand Down
2 changes: 1 addition & 1 deletion integration/commands_diagnostic_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ func TestCommandsDiagnosticGetLog(t *testing.T) {

tt.Parallel()

t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/2775")
t := setup.FailsForSQLite(tt, "https://github.com/FerretDB/FerretDB/issues/3261")

require.NotNil(t, tc.command, "command must not be nil")

Expand Down
2 changes: 1 addition & 1 deletion internal/handlers/sqlite/msg_serverstatus.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ func (h *Handler) MsgServerStatus(ctx context.Context, msg *wire.OpMsg) (*wire.O
}

res.Set("catalogStats", must.NotFail(types.NewDocument(
"collections", int32(0), // TODO
"collections", int32(0), // TODO https://github.com/FerretDB/FerretDB/issues/2775
"capped", int32(0),
"clustered", int32(0),
"timeseries", int32(0),
Expand Down

0 comments on commit eccf40d

Please sign in to comment.