[DP-1714] Ensure tools repo profile script is updated when needed. (#27)

Update the test for the toolrepos step to detect when the profile
script needs to be updated; the previous code did not check for
that when the git repos were already cloned.

VERSION

@@ -1 +1 @@
-0.1.11
+0.1.12

formula/ih-core.rb

@@ -1,5 +1,5 @@
 class IhCore < Formula
-  VERSION="0.1.11"
+  VERSION="0.1.12"
   desc "Brew formula for installing core tools used at Included Health engineering."
   homepage "https://github.com/ConsultingMD/homebrew-ih-public"
   license "CC BY-NC-ND 4.0"

lib/core/toolrepos/step.sh

@@ -56,11 +56,15 @@ function ih::setup::core.toolrepos::test-or-install() {
     git clone git@github.com:ConsultingMD/kore.git --filter=blob:limit=1m --depth=5 "${GR_HOME}/kore" || return
   fi
 
+  local toolsrepo_src_path="$IH_CORE_LIB_DIR/core/toolrepos/default/10_toolrepos.sh"
+  local toolsrepo_tgt_path="$IH_DEFAULT_DIR/10_toolrepos.sh"
+
   if [ "$1" = "test" ]; then
-    return 0
+    ih::file::check-file-in-sync "$toolsrepo_src_path" "$toolsrepo_tgt_path"
+    return
   fi
 
   export IH_WANT_RE_SOURCE=1
 
-  cp -f "$IH_CORE_LIB_DIR/core/toolrepos/default/10_toolrepos.sh" "$IH_DEFAULT_DIR/10_toolrepos.sh"
+  cp -f "$toolsrepo_src_path" "$toolsrepo_tgt_path"
 }

lib/utils/file.sh

@@ -66,6 +66,22 @@ function ih::file::add-if-not-matched() {
   echo "$CONTENT" >>"$FILE"
 }
 
+# Returns 0 if file $2 exists and is the same as file $1. Otherwise returns 1.
+function ih::file::check-file-in-sync() {
+  local SRC=${1:?"src is required"}
+  local DST=${2:?"dst is required"}
+
+  if [ ! -f "$DST" ]; then
+    ih::log::debug "File $DST not found."
+    return 1
+  fi
+
+  if ! diff -q "$DST" "$SRC" >/dev/null; then
+    ih::log::debug "File $DST does not match source"
+    return 1
+  fi
+}
+
 # Returns 0 if the directory at $2 has all
 # the files from the directory at $1, and
 # the files are all the same. Otherwise returns 1.
@@ -76,13 +92,7 @@ function ih::file::check-dir-in-sync() {
 
   for SRC in "$SRC_DIR"/*; do
     local DST="${SRC/$SRC_DIR/$DST_DIR}"
-    if [ ! -f "$DST" ]; then
-      ih::log::debug "File $DST not found."
-      return 1
-    fi
-
-    if ! diff -q "$DST" "$SRC" >/dev/null; then
-      ih::log::debug "File $DST does not match source"
+    if ! ih::file::check-file-in-sync "$SRC" "$DST"; then
       return 1
     fi
   done