Commit Graph

2264 Commits

Author SHA1 Message Date
2ccbb8d587 Store timestamps in DB in Unix time, and display them in the Local time for consistent TZs in displays, and to ensure ordering is correct
This fixes a bug where entries could be out of order if they were recorded on machines in different TZs (e.g. if one machine had the TZ set to UTC). Admittedly, this commit isn't a perfect fix since
it doesn't change all the existing hishtory entries to be stored in UTC, but this just means that the ordering bug will continue to exist for historical entries, but be fixed for new entries. And
for our purposes, that is good enough. :)

 # Please enter the commit message for your changes. Lines starting
2023-09-09 12:28:01 -07:00
223598c8f2 Remove golden that was forked for actions, that didn't need to be 2023-09-09 11:56:06 -07:00
b6b4499a42 Add a few more ExtraDelays + comments to further improve test reliability 2023-09-09 11:50:07 -07:00
4d7e685a6e Add ExtraDelay to make testTui/delete more reliable 2023-09-08 19:44:36 -07:00
392ebfb0fd Fix yaml syntax error introduced in 27d5c59263 2023-09-08 19:41:42 -07:00
ba099c384b Add statsd incr for overall test failure 2023-09-08 19:40:10 -07:00
27d5c59263 Skip datadog integratio on macos if colima fails to start, since colima appears to be flaky due to a code signing error 2023-09-08 19:27:16 -07:00
084af932ea Update docker-compose test to hopefully run and properly record a command in hishtory 2023-09-08 18:05:11 -07:00
3a8243d25d Another set of extra delays for testTui/search, similar to 37c06a31db 2023-09-08 10:09:01 -07:00
cab480b4e6 Merge remote-tracking branch 'origin/master' into sergio/db 2023-09-08 13:08:47 -04:00
cc10689d09 Skip running DD on non-master branches so that PRs can run successfully 2023-09-08 09:59:55 -07:00
37c06a31db Add extra delay before adding token that makes it an invalid search query to make the race condition less likely 2023-09-08 09:53:39 -07:00
3c18f62d99 finishing removing direct DB instructions from http handlers 2023-09-08 10:58:01 -04:00
0d6aa081d8 remove direct db instructions from apiRegisterHandler and statsHandler 2023-09-08 10:58:01 -04:00
382f234c09 remove usage_data related queries from the main package into a database package 2023-09-08 10:58:01 -04:00
294bf78d25 Loosen bash options for docker compose 2023-09-07 22:45:24 -07:00
8db87d2a1e Try customizing the shell for github actions to force it to run as a login shell 2023-09-07 22:37:41 -07:00
3448c3fdf6 Add hishtory export test as a pre-req for testTui/search to ensure that entries are getting properly stored 2023-09-07 22:18:16 -07:00
82d311e3e7 Enable tmate for docker-compose test debugging 2023-09-07 22:12:41 -07:00
519a7ba706 Add OS tag for datadog test export 2023-09-07 22:03:03 -07:00
fe75d67ede Link colima's docker socket to the default location 2023-09-07 22:02:08 -07:00
c67b6b8c82 Install docker on macos github action so the datadog integration works 2023-09-07 22:00:47 -07:00
e5864c9a16 Fix datadog tags to be passed as key/values 2023-09-07 21:05:30 -07:00
2154d9e707 Split docker compose test into two stages 2023-09-07 18:51:13 -07:00
daa4fa2a00 Fix indentation error in go-test.yml 2023-09-07 18:32:17 -07:00
a13549851c Add basic datadog integration to monitor test failures in github actions 2023-09-07 18:30:47 -07:00
9dcc80e9a8 Run hishtory from local dir in docker-compose tests since the shell hasn't been restarted so it isn't yet in the path 2023-09-07 18:29:24 -07:00
81655953b8 Improve the docker compose test by having it check that entries are getting recorded and properly synced 2023-09-07 18:15:47 -07:00
4d0b45d975 Merge pull request #100 from lsmoura/sergio/simple-docker
Improve docker images by using alpine, simplifying Dockerfiles, and improving the docker-compose example
2023-09-07 18:11:32 -07:00
4bc23c9575 Add additional information about sync status to hishtory status -v for debugging purposes 2023-09-07 18:10:17 -07:00
c73b10011b Add TODO for a bug where sorting entries by timestamps in different timezones causes issues 2023-09-07 18:09:30 -07:00
a3e6d5a940 docker improvements:
* use alpine for small docker images
* use docker compose to wait for the database to be ready
* use a single Dockerfile with parameters to determine the correct GOARCH
* update makefile to account for the new GOARCH parameter on the Dockerfile
* remove native-arch-Dockerfile
2023-09-07 11:06:32 -04:00
aa3c7fc41a Follow up to #103: assign err from fmt.Fprintf to _ to ensure go-errcheck doesn't flag it 2023-09-07 07:57:20 -07:00
68e3a813c9 Follow up to #103: pull context from r.Context() when used rather than at the start of functions 2023-09-07 07:56:03 -07:00
86c0acfbc8 Merge pull request #103 from lsmoura/sergio/handlers
Use the existing HTTP request context and modify server to always return a 200 response code even when there is no associated response
2023-09-07 07:48:16 -07:00
e6d922709d fix tests 2023-09-07 09:50:36 -04:00
37670230af Merge remote-tracking branch 'origin/master' into sergio/handlers 2023-09-07 09:33:42 -04:00
589b99e500 do not use http.StatusNoContent 2023-09-07 08:34:21 -04:00
2a95553fdf Fix install.py script for arm7 so installs work on raspberry pis 2023-09-06 21:35:28 -07:00
578113e07c Revert "use errors.Is to determine what the error is" since it doesn't work with modernc.org/sqlite which we need in order to avoid using CGO (which we do to enable easy cross-compiles)
This reverts commit 1589f77956. When building with CGO_ENABLED=0, this code will fail to build.
2023-09-06 20:21:00 -07:00
d80afc27d4 Clear out ~/.other-path/ in tests since it wasn't getting cleared, which meant retries were guaranteed to fail for this function 2023-09-06 20:13:36 -07:00
c5bdee9ad5 Merge pull request #102 from lsmoura/sergio/no-ctx-ptr
Remove context pointers since there is no need to pass around a pointer to an interface
2023-09-06 19:58:08 -07:00
02d04928d5 Merge pull request #101 from lsmoura/sergio/wrap-errors
Wrap errors
2023-09-06 19:56:29 -07:00
2b1ba7e3ba use single context and always return a status to the client
api handlers do not need an extra context. http.Request already has a context that is being ignored, so we leverage it and stop creating a new one.
make the endpoints return http.StatusNoContent instead of just closing the connection from the client.
2023-09-06 11:37:14 -04:00
ce6484f465 constant context keys 2023-09-05 15:54:47 -04:00
a12b0e5f6f no context pointers 2023-09-05 15:45:17 -04:00
1589f77956 use errors.Is to determine what the error is 2023-09-05 15:37:42 -04:00
83ad8c7b1f wrap errors with %w instead of using %v 2023-09-05 15:08:55 -04:00
efa9ddd6df Upgrade checkout action 2023-09-04 21:23:22 -07:00
f778c1b2b6 Increase number of retries for all retriable tests on github actions 2023-09-04 21:21:13 -07:00