mirror of
https://github.com/status-im/status-go.git
synced 2026-08-31 00:51:12 +00:00
Igor Sirotin
f3e4363808
Part of the Go project layout migration, item 27. Pure move plus import-path rewrite across 502 files. No API or behaviour change. `internal/` keeps the messaging application logic unimportable from outside the module, which is what the issue asks for -- status-go is consumed through the C-bindings in mobile/, not as a Go library. Things that had to follow the move, beyond the Go imports: - tools/generate-handlers/template.txt. messenger_handlers.go is generated, and the template hard-codes the imports it emits, so the generated file kept importing protocol/common and failed typecheck. - .gitignore. The ignore rule for that generated file was pinned to the old path; without moving it, a 1486-line generated file starts being tracked. - Makefile: the logosstorage and torrent test targets (both the archive packages and ./protocol itself), the archive README, migration-protocol. - scripts/run_unit_tests.sh, which names the protocol package explicitly to shard its tests. - scripts/cleanup_generated_files.sh and .golangci.yml. scripts/migration_check.sh also needed a fix that is not specific to this move: it validated every file the branch touched under a migration dir against the timestamp naming rule, and a directory rename makes every migration in it look newly added. It now excludes renames, so moving a migration is not mistaken for adding one. refs #7067
60 lines
1.4 KiB
Go
60 lines
1.4 KiB
Go
package protocol
|
|
|
|
import (
|
|
"errors"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
"go.uber.org/zap"
|
|
)
|
|
|
|
func TestWithHistoricSyncInFlightResetsFlagOnError(t *testing.T) {
|
|
m := &Messenger{logger: zap.NewNop()}
|
|
|
|
_, executed, err := m.withHistoricSyncInFlight(func() (*MessengerResponse, error) {
|
|
return nil, errors.New("boom")
|
|
})
|
|
require.Error(t, err)
|
|
require.True(t, executed)
|
|
|
|
m.historicSyncMu.Lock()
|
|
require.False(t, m.historicSyncInFlight)
|
|
m.historicSyncMu.Unlock()
|
|
}
|
|
|
|
func TestWithHistoricSyncInFlightSkipsWhenAlreadyInFlight(t *testing.T) {
|
|
m := &Messenger{logger: zap.NewNop(), historicSyncInFlight: true}
|
|
|
|
called := false
|
|
resp, executed, err := m.withHistoricSyncInFlight(func() (*MessengerResponse, error) {
|
|
called = true
|
|
return &MessengerResponse{}, nil
|
|
})
|
|
require.NoError(t, err)
|
|
require.Nil(t, resp)
|
|
require.False(t, executed)
|
|
require.False(t, called)
|
|
|
|
m.historicSyncMu.Lock()
|
|
require.True(t, m.historicSyncInFlight)
|
|
m.historicSyncMu.Unlock()
|
|
}
|
|
|
|
func TestWithHistoricSyncInFlightRuns(t *testing.T) {
|
|
m := &Messenger{logger: zap.NewNop()}
|
|
|
|
called := false
|
|
resp, executed, err := m.withHistoricSyncInFlight(func() (*MessengerResponse, error) {
|
|
called = true
|
|
return &MessengerResponse{}, nil
|
|
})
|
|
require.NoError(t, err)
|
|
require.NotNil(t, resp)
|
|
require.True(t, executed)
|
|
require.True(t, called)
|
|
|
|
m.historicSyncMu.Lock()
|
|
require.False(t, m.historicSyncInFlight)
|
|
m.historicSyncMu.Unlock()
|
|
}
|