From d60b9310fffa9771de2397daa33e92cc194e710c Mon Sep 17 00:00:00 2001
From: Simon Cropp <simon.cropp@gmail.com>
Date: Mon, 9 Sep 2024 11:33:12 +1000
Subject: [PATCH] add GitAttributes to checks (#1286)

---
 src/Directory.Build.props       |  2 +-
 src/Verify/InnerVerifyChecks.cs | 60 ++++++++++++++++++++++++++-------
 2 files changed, 49 insertions(+), 13 deletions(-)

diff --git a/src/Directory.Build.props b/src/Directory.Build.props
index 42e8f2dbe..9fe2abd2d 100644
--- a/src/Directory.Build.props
+++ b/src/Directory.Build.props
@@ -2,7 +2,7 @@
 <Project>
   <PropertyGroup>
     <NoWarn>CS1591;CS0649;xUnit1026;xUnit1013;CS1573;VerifyTestsProjectDir;VerifySetParameters;PolyFillTargetsForNuget</NoWarn>
-    <Version>26.4.0</Version>
+    <Version>26.4.1</Version>
     <ImplicitUsings>enable</ImplicitUsings>
     <LangVersion>preview</LangVersion>
     <AssemblyVersion>1.0.0</AssemblyVersion>
diff --git a/src/Verify/InnerVerifyChecks.cs b/src/Verify/InnerVerifyChecks.cs
index c6effb904..2ca2a1a03 100644
--- a/src/Verify/InnerVerifyChecks.cs
+++ b/src/Verify/InnerVerifyChecks.cs
@@ -10,6 +10,7 @@ public static async Task Run(Assembly assembly)
         await CheckEditorConfig(directory);
         await CheckGitIgnore(directory);
         await CheckIncorrectlyImportedSnapshots(directory);
+        await CheckGitAttributes(directory);
     }
 
     static async Task CheckIncorrectlyImportedSnapshots(string solutionDirectory)
@@ -51,14 +52,14 @@ This occurs when a test file is copied in the IDE and the IDE incorrectly duplic
 
     static async Task CheckEditorConfig(string solutionDirectory)
     {
-        var editorConfigPath = Path.Combine(solutionDirectory, ".editorconfig");
-        if (!File.Exists(editorConfigPath))
+        var path = Path.Combine(solutionDirectory, ".editorconfig");
+        if (!File.Exists(path))
         {
             return;
         }
 
-        editorConfigPath = Path.GetFullPath(editorConfigPath);
-        var text = await ReadText(editorConfigPath);
+        path = Path.GetFullPath(path);
+        var text = await ReadText(path);
         if (text.Contains("{received,verified}") ||
             text.Contains("# Verify"))
         {
@@ -68,7 +69,7 @@ static async Task CheckEditorConfig(string solutionDirectory)
         throw new(
             $$"""
               Expected .editorconfig to contain settings for Verify.
-              Path: {{editorConfigPath}}
+              Path: {{path}}
               Recommended settings:
 
               # Verify
@@ -84,21 +85,56 @@ static async Task CheckEditorConfig(string solutionDirectory)
               """);
     }
 
+    static async Task CheckGitAttributes(string solutionDirectory)
+    {
+        var path = Path.Combine(solutionDirectory, ".gitattributes");
+        if (!File.Exists(path))
+        {
+            path = Path.Combine(solutionDirectory, "../.gitattributes");
+        }
+
+        if (!File.Exists(path))
+        {
+            return;
+        }
+
+        path = Path.GetFullPath(path);
+        var text = await ReadText(path);
+        if (text.Contains("*.verified.") ||
+            text.Contains("# Verify"))
+        {
+            return;
+        }
+
+        throw new(
+            $"""
+              Expected .gitattributes to contain settings for Verify.
+              Path: {path}
+              Recommended settings:
+
+              # Verify
+              # Extensions should contain all the text files used by snapshots
+              *.verified.txt text eol=lf working-tree-encoding=UTF-8
+              *.verified.xml text eol=lf working-tree-encoding=UTF-8
+              *.verified.json text eol=lf working-tree-encoding=UTF-8
+              """);
+    }
+
     static async Task CheckGitIgnore(string solutionDirectory)
     {
-        var gitIgnorePath = Path.Combine(solutionDirectory, ".gitIgnore");
-        if (!File.Exists(gitIgnorePath))
+        var path = Path.Combine(solutionDirectory, ".gitIgnore");
+        if (!File.Exists(path))
         {
-            gitIgnorePath = Path.Combine(solutionDirectory, "../.gitIgnore");
+            path = Path.Combine(solutionDirectory, "../.gitIgnore");
         }
 
-        if (!File.Exists(gitIgnorePath))
+        if (!File.Exists(path))
         {
             return;
         }
 
-        gitIgnorePath = Path.GetFullPath(gitIgnorePath);
-        var text = await ReadText(gitIgnorePath);
+        path = Path.GetFullPath(path);
+        var text = await ReadText(path);
         if (text.Contains("*.received.*") ||
             text.Contains("*.received/") ||
             text.Contains("# Verify"))
@@ -109,7 +145,7 @@ static async Task CheckGitIgnore(string solutionDirectory)
         throw new(
             $"""
               Expected .gitIgnore to contain settings for Verify.
-              Path: {gitIgnorePath}
+              Path: {path}
               Recommended settings:
 
               # Verify