Просмотр исходного кода

Removed an issue pertaining to tests

arianagiroux 1 неделя назад
Родитель
Сommit
5eaef26a14

+ 0 - 0
issues/tests: cant test scroll percent?/blockedby/viewport-v2?


+ 0 - 12
issues/tests: cant test scroll percent?/description

@@ -1,12 +0,0 @@
-Um... I don't know whats going on but when i try to go to viewport v2 it breaks
-
----
-I think this might specifically pertain to the V1/V2 version - and is also somewhat beyond the scope of our tests. Should our tests simply assert that the framework is truth and will work?
-
---- @023759d
-I've attached a patch based on the above commit that outlines the issues. Try running the tui tests, it should fail on the final test (update on j).
-
-For the patch, see the patch file in the issue dir
-
----
-theoretically we could test against the view changing, instead of anything relating to the actual scrolling data

+ 0 - 198
issues/tests: cant test scroll percent?/reproduce.patch.023759d

@@ -1,198 +0,0 @@
-diff --git a/go.mod b/go.mod
-index fdfd67a..9b6b59c 100644
---- a/go.mod
-+++ b/go.mod
-@@ -4,7 +4,7 @@ go 1.25.6
- 
- require (
- 	github.com/NimbleMarkets/ntcharts v0.4.0
--	github.com/charmbracelet/bubbles v0.20.0
-+	github.com/charmbracelet/bubbles v1.0.0
- 	github.com/charmbracelet/bubbletea v1.3.10
- 	github.com/charmbracelet/lipgloss v1.1.0
- 	github.com/stretchr/testify v1.11.1
-@@ -12,24 +12,27 @@ require (
- 
- require (
- 	github.com/aymanbagabas/go-osc52/v2 v2.0.1 // indirect
--	github.com/charmbracelet/colorprofile v0.2.3-0.20250311203215-f60798e515dc // indirect
--	github.com/charmbracelet/x/ansi v0.10.1 // indirect
--	github.com/charmbracelet/x/cellbuf v0.0.13-0.20250311204145-2c3ea96c31dd // indirect
--	github.com/charmbracelet/x/term v0.2.1 // indirect
-+	github.com/charmbracelet/colorprofile v0.4.1 // indirect
-+	github.com/charmbracelet/x/ansi v0.11.6 // indirect
-+	github.com/charmbracelet/x/cellbuf v0.0.15 // indirect
-+	github.com/charmbracelet/x/term v0.2.2 // indirect
-+	github.com/clipperhouse/displaywidth v0.9.0 // indirect
-+	github.com/clipperhouse/stringish v0.1.1 // indirect
-+	github.com/clipperhouse/uax29/v2 v2.5.0 // indirect
- 	github.com/davecgh/go-spew v1.1.1 // indirect
- 	github.com/erikgeiser/coninput v0.0.0-20211004153227-1c3628e74d0f // indirect
- 	github.com/lrstanley/bubblezone v0.0.0-20240914071701-b48c55a5e78e // indirect
--	github.com/lucasb-eyer/go-colorful v1.2.0 // indirect
-+	github.com/lucasb-eyer/go-colorful v1.3.0 // indirect
- 	github.com/mattn/go-isatty v0.0.20 // indirect
- 	github.com/mattn/go-localereader v0.0.1 // indirect
--	github.com/mattn/go-runewidth v0.0.16 // indirect
-+	github.com/mattn/go-runewidth v0.0.19 // indirect
- 	github.com/muesli/ansi v0.0.0-20230316100256-276c6243b2f6 // indirect
- 	github.com/muesli/cancelreader v0.2.2 // indirect
- 	github.com/muesli/termenv v0.16.0 // indirect
- 	github.com/pmezard/go-difflib v1.0.0 // indirect
- 	github.com/rivo/uniseg v0.4.7 // indirect
- 	github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e // indirect
--	golang.org/x/sys v0.36.0 // indirect
-+	golang.org/x/sys v0.38.0 // indirect
- 	golang.org/x/text v0.20.0 // indirect
- 	gopkg.in/yaml.v3 v3.0.1 // indirect
- )
-diff --git a/go.sum b/go.sum
-index 9bd8124..b8fcbb6 100644
---- a/go.sum
-+++ b/go.sum
-@@ -2,34 +2,40 @@ github.com/NimbleMarkets/ntcharts v0.4.0 h1:BtrER5o6s3xMAebhSDQZpdFdfVMGMpV4Qz8l
- github.com/NimbleMarkets/ntcharts v0.4.0/go.mod h1:zVeRqYkh2n59YPe1bflaSL4O2aD2ZemNmrbdEqZ70hk=
- github.com/aymanbagabas/go-osc52/v2 v2.0.1 h1:HwpRHbFMcZLEVr42D4p7XBqjyuxQH5SMiErDT4WkJ2k=
- github.com/aymanbagabas/go-osc52/v2 v2.0.1/go.mod h1:uYgXzlJ7ZpABp8OJ+exZzJJhRNQ2ASbcXHWsFqH8hp8=
--github.com/charmbracelet/bubbles v0.20.0 h1:jSZu6qD8cRQ6k9OMfR1WlM+ruM8fkPWkHvQWD9LIutE=
--github.com/charmbracelet/bubbles v0.20.0/go.mod h1:39slydyswPy+uVOHZ5x/GjwVAFkCsV8IIVy+4MhzwwU=
-+github.com/charmbracelet/bubbles v1.0.0 h1:12J8/ak/uCZEMQ6KU7pcfwceyjLlWsDLAxB5fXonfvc=
-+github.com/charmbracelet/bubbles v1.0.0/go.mod h1:9d/Zd5GdnauMI5ivUIVisuEm3ave1XwXtD1ckyV6r3E=
- github.com/charmbracelet/bubbletea v1.3.10 h1:otUDHWMMzQSB0Pkc87rm691KZ3SWa4KUlvF9nRvCICw=
- github.com/charmbracelet/bubbletea v1.3.10/go.mod h1:ORQfo0fk8U+po9VaNvnV95UPWA1BitP1E0N6xJPlHr4=
--github.com/charmbracelet/colorprofile v0.2.3-0.20250311203215-f60798e515dc h1:4pZI35227imm7yK2bGPcfpFEmuY1gc2YSTShr4iJBfs=
--github.com/charmbracelet/colorprofile v0.2.3-0.20250311203215-f60798e515dc/go.mod h1:X4/0JoqgTIPSFcRA/P6INZzIuyqdFY5rm8tb41s9okk=
-+github.com/charmbracelet/colorprofile v0.4.1 h1:a1lO03qTrSIRaK8c3JRxJDZOvhvIeSco3ej+ngLk1kk=
-+github.com/charmbracelet/colorprofile v0.4.1/go.mod h1:U1d9Dljmdf9DLegaJ0nGZNJvoXAhayhmidOdcBwAvKk=
- github.com/charmbracelet/lipgloss v1.1.0 h1:vYXsiLHVkK7fp74RkV7b2kq9+zDLoEU4MZoFqR/noCY=
- github.com/charmbracelet/lipgloss v1.1.0/go.mod h1:/6Q8FR2o+kj8rz4Dq0zQc3vYf7X+B0binUUBwA0aL30=
--github.com/charmbracelet/x/ansi v0.10.1 h1:rL3Koar5XvX0pHGfovN03f5cxLbCF2YvLeyz7D2jVDQ=
--github.com/charmbracelet/x/ansi v0.10.1/go.mod h1:3RQDQ6lDnROptfpWuUVIUG64bD2g2BgntdxH0Ya5TeE=
--github.com/charmbracelet/x/cellbuf v0.0.13-0.20250311204145-2c3ea96c31dd h1:vy0GVL4jeHEwG5YOXDmi86oYw2yuYUGqz6a8sLwg0X8=
--github.com/charmbracelet/x/cellbuf v0.0.13-0.20250311204145-2c3ea96c31dd/go.mod h1:xe0nKWGd3eJgtqZRaN9RjMtK7xUYchjzPr7q6kcvCCs=
--github.com/charmbracelet/x/term v0.2.1 h1:AQeHeLZ1OqSXhrAWpYUtZyX1T3zVxfpZuEQMIQaGIAQ=
--github.com/charmbracelet/x/term v0.2.1/go.mod h1:oQ4enTYFV7QN4m0i9mzHrViD7TQKvNEEkHUMCmsxdUg=
-+github.com/charmbracelet/x/ansi v0.11.6 h1:GhV21SiDz/45W9AnV2R61xZMRri5NlLnl6CVF7ihZW8=
-+github.com/charmbracelet/x/ansi v0.11.6/go.mod h1:2JNYLgQUsyqaiLovhU2Rv/pb8r6ydXKS3NIttu3VGZQ=
-+github.com/charmbracelet/x/cellbuf v0.0.15 h1:ur3pZy0o6z/R7EylET877CBxaiE1Sp1GMxoFPAIztPI=
-+github.com/charmbracelet/x/cellbuf v0.0.15/go.mod h1:J1YVbR7MUuEGIFPCaaZ96KDl5NoS0DAWkskup+mOY+Q=
-+github.com/charmbracelet/x/term v0.2.2 h1:xVRT/S2ZcKdhhOuSP4t5cLi5o+JxklsoEObBSgfgZRk=
-+github.com/charmbracelet/x/term v0.2.2/go.mod h1:kF8CY5RddLWrsgVwpw4kAa6TESp6EB5y3uxGLeCqzAI=
-+github.com/clipperhouse/displaywidth v0.9.0 h1:Qb4KOhYwRiN3viMv1v/3cTBlz3AcAZX3+y9OLhMtAtA=
-+github.com/clipperhouse/displaywidth v0.9.0/go.mod h1:aCAAqTlh4GIVkhQnJpbL0T/WfcrJXHcj8C0yjYcjOZA=
-+github.com/clipperhouse/stringish v0.1.1 h1:+NSqMOr3GR6k1FdRhhnXrLfztGzuG+VuFDfatpWHKCs=
-+github.com/clipperhouse/stringish v0.1.1/go.mod h1:v/WhFtE1q0ovMta2+m+UbpZ+2/HEXNWYXQgCt4hdOzA=
-+github.com/clipperhouse/uax29/v2 v2.5.0 h1:x7T0T4eTHDONxFJsL94uKNKPHrclyFI0lm7+w94cO8U=
-+github.com/clipperhouse/uax29/v2 v2.5.0/go.mod h1:Wn1g7MK6OoeDT0vL+Q0SQLDz/KpfsVRgg6W7ihQeh4g=
- github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
- github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
- github.com/erikgeiser/coninput v0.0.0-20211004153227-1c3628e74d0f h1:Y/CXytFA4m6baUTXGLOoWe4PQhGxaX0KpnayAqC48p4=
- github.com/erikgeiser/coninput v0.0.0-20211004153227-1c3628e74d0f/go.mod h1:vw97MGsxSvLiUE2X8qFplwetxpGLQrlU1Q9AUEIzCaM=
- github.com/lrstanley/bubblezone v0.0.0-20240914071701-b48c55a5e78e h1:OLwZ8xVaeVrru0xyeuOX+fne0gQTFEGlzfNjipCbxlU=
- github.com/lrstanley/bubblezone v0.0.0-20240914071701-b48c55a5e78e/go.mod h1:NQ34EGeu8FAYGBMDzwhfNJL8YQYoWZP5xYJPRDAwN3E=
--github.com/lucasb-eyer/go-colorful v1.2.0 h1:1nnpGOrhyZZuNyfu1QjKiUICQ74+3FNCN69Aj6K7nkY=
--github.com/lucasb-eyer/go-colorful v1.2.0/go.mod h1:R4dSotOR9KMtayYi1e77YzuveK+i7ruzyGqttikkLy0=
-+github.com/lucasb-eyer/go-colorful v1.3.0 h1:2/yBRLdWBZKrf7gB40FoiKfAWYQ0lqNcbuQwVHXptag=
-+github.com/lucasb-eyer/go-colorful v1.3.0/go.mod h1:R4dSotOR9KMtayYi1e77YzuveK+i7ruzyGqttikkLy0=
- github.com/mattn/go-isatty v0.0.20 h1:xfD0iDuEKnDkl03q4limB+vH+GxLEtL/jb4xVJSWWEY=
- github.com/mattn/go-isatty v0.0.20/go.mod h1:W+V8PltTTMOvKvAeJH7IuucS94S2C6jfK/D7dTCTo3Y=
- github.com/mattn/go-localereader v0.0.1 h1:ygSAOl7ZXTx4RdPYinUpg6W99U8jWvWi9Ye2JC/oIi4=
- github.com/mattn/go-localereader v0.0.1/go.mod h1:8fBrzywKY7BI3czFoHkuzRoWE9C+EiG4R1k4Cjx5p88=
--github.com/mattn/go-runewidth v0.0.16 h1:E5ScNMtiwvlvB5paMFdw9p4kSQzbXFikJ5SQO6TULQc=
--github.com/mattn/go-runewidth v0.0.16/go.mod h1:Jdepj2loyihRzMpdS35Xk/zdY8IAYHsh153qUoGf23w=
-+github.com/mattn/go-runewidth v0.0.19 h1:v++JhqYnZuu5jSKrk9RbgF5v4CGUjqRfBm05byFGLdw=
-+github.com/mattn/go-runewidth v0.0.19/go.mod h1:XBkDxAl56ILZc9knddidhrOlY5R/pDhgLpndooCuJAs=
- github.com/muesli/ansi v0.0.0-20230316100256-276c6243b2f6 h1:ZK8zHtRHOkbHy6Mmr5D264iyp3TiX5OmNcI5cIARiQI=
- github.com/muesli/ansi v0.0.0-20230316100256-276c6243b2f6/go.mod h1:CJlz5H+gyd6CUWT45Oy4q24RdLyn7Md9Vj2/ldJBSIo=
- github.com/muesli/cancelreader v0.2.2 h1:3I4Kt4BQjOR54NavqnDogx/MIoWBFa0StPA8ELUXHmA=
-@@ -38,19 +44,18 @@ github.com/muesli/termenv v0.16.0 h1:S5AlUN9dENB57rsbnkPyfdGuWIlkmzJjbFf0Tf5FWUc
- github.com/muesli/termenv v0.16.0/go.mod h1:ZRfOIKPFDYQoDFF4Olj7/QJbW60Ol/kL1pU3VfY/Cnk=
- github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
- github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
--github.com/rivo/uniseg v0.2.0/go.mod h1:J6wj4VEh+S6ZtnVlnTBMWIodfgj8LQOQFoIToxlJtxc=
- github.com/rivo/uniseg v0.4.7 h1:WUdvkW8uEhrYfLC4ZzdpI2ztxP1I582+49Oc5Mq64VQ=
- github.com/rivo/uniseg v0.4.7/go.mod h1:FN3SvrM+Zdj16jyLfmOkMNblXMcoc8DfTHruCPUcx88=
- github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
- github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
- github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e h1:JVG44RsyaB9T2KIHavMF/ppJZNG9ZpyihvCd0w101no=
- github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e/go.mod h1:RbqR21r5mrJuqunuUZ/Dhy/avygyECGrLceyNeo4LiM=
--golang.org/x/exp v0.0.0-20220909182711-5c715a9e8561 h1:MDc5xs78ZrZr3HMQugiXOAkSZtfTpbJLDr/lwfgO53E=
--golang.org/x/exp v0.0.0-20220909182711-5c715a9e8561/go.mod h1:cyybsKvd6eL0RnXn6p/Grxp8F5bW7iYuBgsNCOHpMYE=
-+golang.org/x/exp v0.0.0-20231006140011-7918f672742d h1:jtJma62tbqLibJ5sFQz8bKtEM8rJBtfilJ2qTU199MI=
-+golang.org/x/exp v0.0.0-20231006140011-7918f672742d/go.mod h1:ldy0pHrwJyGW56pPQzzkH36rKxoZW1tw7ZJpeKx+hdo=
- golang.org/x/sys v0.0.0-20210809222454-d867a43fc93e/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
- golang.org/x/sys v0.6.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
--golang.org/x/sys v0.36.0 h1:KVRy2GtZBrk1cBYA7MKu5bEZFxQk4NIDV6RLVcC8o0k=
--golang.org/x/sys v0.36.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks=
-+golang.org/x/sys v0.38.0 h1:3yZWxaJjBmCWXqhN1qh02AkOnCQ1poK6oF+a7xWL6Gc=
-+golang.org/x/sys v0.38.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks=
- golang.org/x/text v0.20.0 h1:gK/Kv2otX8gz+wn7Rmb3vT96ZwuoxnQlY+HlJVj7Qug=
- golang.org/x/text v0.20.0/go.mod h1:D4IsuqiFMhST5bX19pQ9ikHC2GsaKyk/oF+pn3ducp4=
- gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
-diff --git a/tui.go b/tui.go
-index d362feb..4650a4b 100644
---- a/tui.go
-+++ b/tui.go
-@@ -37,6 +37,7 @@ var (
- 
- type ( // tea.Msg signatures
- 	tickMsg       time.Time
-+	scrollMsg     int // positive = scrolled up, negative = scrolled down
- 	pollResultMsg struct {
- 		results []float64
- 		index   int
-@@ -56,7 +57,7 @@ type Model struct {
- func InitialModel(addresses []string, speed time.Duration, chartHeight int) Model {
- 	var model Model
- 	model.viewport = viewport.New(0, 0)
--	model.viewport.MouseWheelEnabled = true
-+	// model.viewport.MouseWheelEnabled = true
- 	model.UpdateSpeed = speed
- 	model.ChartHeight = chartHeight
- 
-@@ -143,9 +144,12 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
- 	case tea.KeyMsg:
- 		if k := msg.String(); k == "j" { // scroll down
- 			m.viewport.LineDown(8)
-+			cmd := func() tea.Msg { return scrollMsg(1) }
-+			cmds = append(cmds, cmd)
- 		} else if k == "k" { // scroll up
- 			m.viewport.LineUp(8)
--		} else {
-+			cmds = append(cmds, func() tea.Msg { return scrollMsg(-1) })
-+		} else if k == "ctrl+c" || k == "q" {
- 			return m, tea.Quit
- 		}
- 
-diff --git a/tui_test.go b/tui_test.go
-index 9887ecd..fdb377f 100644
---- a/tui_test.go
-+++ b/tui_test.go
-@@ -46,11 +46,10 @@ func Test_Model_Update_window_resize(t *testing.T) {
- }
- 
- func Test_Model_Update_keymsg_j(t *testing.T) {
--	t.Skip()
- 	testKey := tea.Key{Type: tea.KeyRunes, Runes: []rune{'j'}}
- 	testMsg := tea.KeyMsg(testKey)
- 	model := tea.Model(InitialModel([]string{"127.0.0.1", "127.0.0.1"}, time.Duration(time.Second*1), 0))
--	scrollPercent := model.(Model).viewport.ScrollPercent()
-+
- 	var msgs []tea.Msg
- 	var cmd tea.Cmd
- 	model, cmd = model.Update(tea.WindowSizeMsg{Width: 100, Height: 2000})
-@@ -58,8 +57,12 @@ func Test_Model_Update_keymsg_j(t *testing.T) {
- 		msgs = append(msgs, cmd())
- 	}
- 	msgs = append(msgs, testMsg)
--	model, _ = model.Update(msgs)
--	assert.NotEqual(t, scrollPercent, model.(Model).viewport.ScrollPercent())
-+	var m tea.Cmd
-+	_, m = model.Update(msgs)
-+	assert.NotNil(t, m)
-+	// value := m()
-+	// assert.IsType(t, new(scrollMsg), value)
-+	// assert.Equal(t, 1, value)
- }
- 
- func Test_Model_View(t *testing.T)    {}

+ 0 - 1
issues/tests: cant test scroll percent?/status

@@ -1 +0,0 @@
-open

+ 0 - 0
issues/tests: cant test scroll percent?/tags/tests


+ 0 - 0
issues/tests: cant test scroll percent?/tags/v0.1