From 5128d3a1eaa741864dac3771aafb0b5326e33b76 Mon Sep 17 00:00:00 2001
From: aszlig <aszlig@nix.build>
Date: Tue, 10 Nov 2020 08:16:03 +0100
Subject: games/starbound: Update to version 1.4.4

I already updated the game a while ago, but didn't commit it yet and
since the update has been out since a year, I think it's time to
actually update it here as well (shame on me).

This essentially includes the "new" bounty hunter update from June 2019,
which has a lot of changes.

So instead of including the changelog here, I'm hereby referring to the
upstream release announcement:

https://playstarbound.com/starbound-1-4-bounty-hunter-update/

I also changed the -Werror flag in the preloader to -Wall, since the
errors/warnings we're getting are false-positives:

  error: '__builtin_strncpy' specified bound depends on the length of
  the source argument [-Werror=stringop-overflow=]

While a warning like this usually is a concern, it's not in our case,
since the size of the destination *also* depends on the length of the
source argument.

Additionally, I changed the test to use the new
networking.interfaces.*.ipv4.addresses option instead of the legacy one
which has been deprecated for ages.

Signed-off-by: aszlig <aszlig@nix.build>
---
 tests/games/starbound.nix | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

(limited to 'tests')

diff --git a/tests/games/starbound.nix b/tests/games/starbound.nix
index 5fbb85aa..02b4466f 100644
--- a/tests/games/starbound.nix
+++ b/tests/games/starbound.nix
@@ -10,9 +10,8 @@ let
     '';
     escapeScreenshot = pkgs.lib.replaceStrings ["-"] ["_"];
   in ''
-    with client.nested("${description}"):
-      client.screenshot("before_${escapeScreenshot name}")
-      client.succeed("${pkgs.xdotool}/bin/xdotool '${xdoFile}'")
+    client.screenshot("before_${escapeScreenshot name}")
+    client.succeed("${pkgs.xdotool}/bin/xdotool '${xdoFile}'")
   '';
 
   clickAt = name: x: y: xdo {
@@ -38,7 +37,7 @@ in {
   enableOCR = true;
 
   nodes = {
-    server = {
+    server = { lib, ... }: {
       vuizvui.services.starbound = {
         enable = true;
         # Use a different dataDir than the default to make
@@ -47,12 +46,14 @@ in {
         users.alice.password = "secret";
       };
       virtualisation.memorySize = 2047;
-      networking.interfaces.eth1.ipAddress = "192.168.0.1";
-      networking.interfaces.eth1.prefixLength = 24;
+      networking.interfaces.eth1.ipv4.addresses = lib.singleton {
+        address = "192.168.0.1";
+        prefixLength = 24;
+      };
       networking.firewall.enable = false;
     };
 
-    client = { pkgs, ... }: {
+    client = { lib, pkgs, ... }: {
       imports = [
         "${nixpkgsPath}/nixos/tests/common/x11.nix"
       ];
@@ -60,8 +61,10 @@ in {
       environment.systemPackages = [
         pkgs.vuizvui.games.humblebundle.starbound
       ];
-      networking.interfaces.eth1.ipAddress = "192.168.0.2";
-      networking.interfaces.eth1.prefixLength = 24;
+      networking.interfaces.eth1.ipv4.addresses = lib.singleton {
+        address = "192.168.0.2";
+        prefixLength = 24;
+      };
       networking.firewall.enable = false;
     };
   };
@@ -70,10 +73,9 @@ in {
     # fmt: off
     server.wait_for_unit("starbound.service")
 
-    with client.nested("waiting for client to start up"):
-      client.wait_for_x()
-      client.succeed("starbound >&2 &")
-      client.wait_for_text('(?i)options')
+    client.wait_for_x()
+    client.succeed("starbound >&2 &")
+    client.wait_for_text('(?i)options')
 
     ${clickAt "join-game" 100 560}
     client.wait_for_text('(?i)select')
-- 
cgit 1.4.1