From ac6ed3d939fe9dad81aadbd04541e905c625ca82 Mon Sep 17 00:00:00 2001 From: tobi <31960611+tsmethurst@users.noreply.github.com> Date: Mon, 15 Aug 2022 12:35:05 +0200 Subject: [chore] Update bun / sqlite versions; update gtsmodels (#754) * upstep bun and sqlite versions * allow specific columns to be updated in the db * only update necessary columns for user * bit tidier * only update necessary fields of media_attachment * only update relevant instance fields * update tests * update only specific account columns * use bool pointers on gtsmodels includes attachment, status, account, user * update columns more selectively * test all default fields on new account insert * updating remaining bools on gtsmodels * initialize pointer fields when extracting AP emoji * copy bools properly * add copyBoolPtr convenience function + test it * initialize false bool ptrs a bit more neatly --- internal/federation/dereferencing/status_test.go | 38 ++++++++++++------------ 1 file changed, 19 insertions(+), 19 deletions(-) (limited to 'internal/federation/dereferencing/status_test.go') diff --git a/internal/federation/dereferencing/status_test.go b/internal/federation/dereferencing/status_test.go index 6f6c26145..964de23ec 100644 --- a/internal/federation/dereferencing/status_test.go +++ b/internal/federation/dereferencing/status_test.go @@ -46,7 +46,7 @@ func (suite *StatusTestSuite) TestDereferenceSimpleStatus() { suite.Equal("https://unknown-instance.com/users/@brand_new_person/01FE4NTHKWW7THT67EF10EB839", status.URL) suite.Equal("Hello world!", status.Content) suite.Equal("https://unknown-instance.com/users/brand_new_person", status.AccountURI) - suite.False(status.Local) + suite.False(*status.Local) suite.Empty(status.ContentWarning) suite.Equal(gtsmodel.VisibilityPublic, status.Visibility) suite.Equal(ap.ObjectNote, status.ActivityStreamsType) @@ -55,16 +55,16 @@ func (suite *StatusTestSuite) TestDereferenceSimpleStatus() { dbStatus, err := suite.db.GetStatusByURI(context.Background(), status.URI) suite.NoError(err) suite.Equal(status.ID, dbStatus.ID) - suite.True(dbStatus.Federated) - suite.True(dbStatus.Boostable) - suite.True(dbStatus.Replyable) - suite.True(dbStatus.Likeable) + suite.True(*dbStatus.Federated) + suite.True(*dbStatus.Boostable) + suite.True(*dbStatus.Replyable) + suite.True(*dbStatus.Likeable) // account should be in the database now too account, err := suite.db.GetAccountByURI(context.Background(), status.AccountURI) suite.NoError(err) suite.NotNil(account) - suite.True(account.Discoverable) + suite.True(*account.Discoverable) suite.Equal("https://unknown-instance.com/users/brand_new_person", account.URI) suite.Equal("hey I'm a new person, your instance hasn't seen me yet uwu", account.Note) suite.Equal("Geoff Brando New Personson", account.DisplayName) @@ -86,7 +86,7 @@ func (suite *StatusTestSuite) TestDereferenceStatusWithMention() { suite.Equal("https://unknown-instance.com/users/@brand_new_person/01FE5Y30E3W4P7TRE0R98KAYQV", status.URL) suite.Equal("Hey @the_mighty_zork@localhost:8080 how's it going?", status.Content) suite.Equal("https://unknown-instance.com/users/brand_new_person", status.AccountURI) - suite.False(status.Local) + suite.False(*status.Local) suite.Empty(status.ContentWarning) suite.Equal(gtsmodel.VisibilityPublic, status.Visibility) suite.Equal(ap.ObjectNote, status.ActivityStreamsType) @@ -95,16 +95,16 @@ func (suite *StatusTestSuite) TestDereferenceStatusWithMention() { dbStatus, err := suite.db.GetStatusByURI(context.Background(), status.URI) suite.NoError(err) suite.Equal(status.ID, dbStatus.ID) - suite.True(dbStatus.Federated) - suite.True(dbStatus.Boostable) - suite.True(dbStatus.Replyable) - suite.True(dbStatus.Likeable) + suite.True(*dbStatus.Federated) + suite.True(*dbStatus.Boostable) + suite.True(*dbStatus.Replyable) + suite.True(*dbStatus.Likeable) // account should be in the database now too account, err := suite.db.GetAccountByURI(context.Background(), status.AccountURI) suite.NoError(err) suite.NotNil(account) - suite.True(account.Discoverable) + suite.True(*account.Discoverable) suite.Equal("https://unknown-instance.com/users/brand_new_person", account.URI) suite.Equal("hey I'm a new person, your instance hasn't seen me yet uwu", account.Note) suite.Equal("Geoff Brando New Personson", account.DisplayName) @@ -121,7 +121,7 @@ func (suite *StatusTestSuite) TestDereferenceStatusWithMention() { suite.Equal(account.ID, m.OriginAccountID) suite.Equal(fetchingAccount.ID, m.TargetAccountID) suite.Equal(account.URI, m.OriginAccountURI) - suite.False(m.Silent) + suite.False(*m.Silent) } func (suite *StatusTestSuite) TestDereferenceStatusWithImageAndNoContent() { @@ -137,7 +137,7 @@ func (suite *StatusTestSuite) TestDereferenceStatusWithImageAndNoContent() { suite.Equal("https://turnip.farm/@turniplover6969/70c53e54-3146-42d5-a630-83c8b6c7c042", status.URL) suite.Equal("", status.Content) suite.Equal("https://turnip.farm/users/turniplover6969", status.AccountURI) - suite.False(status.Local) + suite.False(*status.Local) suite.Empty(status.ContentWarning) suite.Equal(gtsmodel.VisibilityPublic, status.Visibility) suite.Equal(ap.ObjectNote, status.ActivityStreamsType) @@ -146,16 +146,16 @@ func (suite *StatusTestSuite) TestDereferenceStatusWithImageAndNoContent() { dbStatus, err := suite.db.GetStatusByURI(context.Background(), status.URI) suite.NoError(err) suite.Equal(status.ID, dbStatus.ID) - suite.True(dbStatus.Federated) - suite.True(dbStatus.Boostable) - suite.True(dbStatus.Replyable) - suite.True(dbStatus.Likeable) + suite.True(*dbStatus.Federated) + suite.True(*dbStatus.Boostable) + suite.True(*dbStatus.Replyable) + suite.True(*dbStatus.Likeable) // account should be in the database now too account, err := suite.db.GetAccountByURI(context.Background(), status.AccountURI) suite.NoError(err) suite.NotNil(account) - suite.True(account.Discoverable) + suite.True(*account.Discoverable) suite.Equal("https://turnip.farm/users/turniplover6969", account.URI) suite.Equal("I just think they're neat", account.Note) suite.Equal("Turnip Lover 6969", account.DisplayName) -- cgit v1.2.3