lipgloss icon indicating copy to clipboard operation
lipgloss copied to clipboard

Add TestPlace for position.go

Open bashbunni opened this issue 3 years ago • 2 comments

Changes:

  • [x] add test for Place from position.go

Upgrades:

  • increase test coverage for position.go from 0% to 70.91%

bashbunni avatar Apr 11 '22 17:04 bashbunni

@bashbunni I think this is good to merge, one thing I would recommend is removing the require dependency and simply using go == for string comparison.

maaslalani avatar Mar 14 '24 15:03 maaslalani

I'll rebase and fix the conflicts, then see if it still works :joy:

bashbunni avatar Mar 15 '24 13:03 bashbunni

This should be covered in a test, but I'm going to close this one as I'd prefer to use a golden file for this

bashbunni avatar Aug 20 '24 00:08 bashbunni