ts2 / ts2-sim-server

TS2 SimServer, the core of the TS2 Simulator
https://ts2.github.io
GNU General Public License v2.0
5 stars 9 forks source link

Fix crash on simulation loading caused by missing signal actions list #49

Open orwell96 opened 3 years ago

orwell96 commented 3 years ago

As title says, this fixes a crash when loading a previously saved simulation It appears to happen as a result of the signal actions not being saved in the JSON file.

OT: You are probably no longer maintaining and continuing this project. I don't expect you to actually review and merge this. As I am actually pretty impressed by the work you put into this project, and am sad to see this lie around unfinished, I have decided to fork and continue it, although starting off from 0.6. If you're interested: https://github.com/orwell96/ts2-fork

codecov[bot] commented 3 years ago

Codecov Report

Merging #49 (7d786e4) into master (97df100) will decrease coverage by 2.49%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
- Coverage   76.26%   73.77%   -2.50%     
==========================================
  Files          38       38              
  Lines        2600     2208     -392     
==========================================
- Hits         1983     1629     -354     
+ Misses        494      459      -35     
+ Partials      123      120       -3     
Impacted Files Coverage Δ
simulation/trains.go 55.91% <0.00%> (-4.96%) :arrow_down:
server/requests.go 75.00% <0.00%> (-6.82%) :arrow_down:
simulation/color.go 43.47% <0.00%> (-6.53%) :arrow_down:
plugins/routes/routes.go 70.00% <0.00%> (-6.00%) :arrow_down:
simulation/signal_conditions.go 58.18% <0.00%> (-5.09%) :arrow_down:
plugins/points/points.go 64.28% <0.00%> (-4.14%) :arrow_down:
simulation/track_signals.go 77.71% <0.00%> (-3.94%) :arrow_down:
server/websocket.go 75.00% <0.00%> (-3.58%) :arrow_down:
server/connection.go 73.58% <0.00%> (-3.09%) :arrow_down:
... and 29 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 97df100...7d786e4. Read the comment docs.