Hochbaustatistik/config/Jenkinsfile aktualisiert

Steps und Environment angepasst

Hochbaustatistik.Testing/DatabaseOperationsTests.cs

@@ -12,7 +12,7 @@ namespace Hochbaustatistik.Testing
     public class DatabaseOperationsTests
     {
 
-        [Fact]//(Skip ="Keine Datenbank-Verbindung m<>glich")]
+        [Fact (Skip ="Keine Datenbank-Verbindung m<>glich")]
         public void ConnectAndExecuteHochbauTest()
         {           
             DatabaseOperations.conString = "Data Source=(DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=dborap04.stadtdo.de)(PORT=1521))(CONNECT_DATA=(SERVICE_NAME=bgvp.stadtdo.de)));user id=leser;password=leser;";

Hochbaustatistik.Testing/EmailTests.cs

@@ -11,7 +11,7 @@ namespace Hochbaustatistik.Testing
 {
     public class EmailTests
     {
-        [Theory(Skip = "Kein lotus installiert")]      
+        [Theory(Skip = "Kein Lotus Notes installiert")]      
         [InlineData(@"C:\TEMP\test.txt")]
         public void SenMailTest(string attachment)
         {           

Hochbaustatistik.Testing/Utilities/DataOpsTest.cs

@@ -14,7 +14,6 @@ namespace Hochbaustatistik.Testing.Utilities
 {
     public class DataOpsTest
     {       
-        readonly string sourceFile = @"TestData\TestData.csv";
         public readonly VorgangInpro vorgang = new VorgangInpro() { Schlussabnahme = "2022" };
                
         [Theory]
@@ -119,7 +118,7 @@ namespace Hochbaustatistik.Testing.Utilities
             Assert.Equal(expected2, actuals.ElementAt(1));
         }
 
-        [Fact]
+        [Fact(Skip = "Keine Datenbank-Verbindung möglich")]
         public void MergeInproWithITNRWTest()
         {
             var resultFile = @"C:\TEMP\hochbaustatistik.csv";

Hochbaustatistik.Testing/WebTesting.cs

@@ -20,7 +20,7 @@ namespace Hochbaustatistik.Testing
     public class WebTesting
     {   
 
-        [Fact (Skip ="Keine Daten zum Download vorhanden")]
+        [Fact (Skip ="Derzeit keine Daten zum Download vorhanden")]
         public void DownloadList()
         {
             //DownloadFolder
@@ -58,7 +58,7 @@ namespace Hochbaustatistik.Testing
             driver.Quit();
 
             Assert.True(File.Exists(pathDownloadedList));
-            //File.Delete(pathDownloadedList);
+            File.Delete(pathDownloadedList);
         }        
     }
 }

Hochbaustatistik/Business/Vorgang.cs

@@ -60,7 +60,7 @@ namespace Hochbaustatistik.Business
         #region Methods
         string ReplaceHiphenWithSlash(string aktenzeichen)
         {
-            var regex = new Regex(Regex.Escape("-"));
+            var regex = new Regex(Regex.Escape("-"),RegexOptions.None,TimeSpan.FromMilliseconds(5));
             string res = regex.Replace(aktenzeichen, "/", 1);
             return res;
         }

Hochbaustatistik/Business/VorgangInpro.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Linq;
 using System.Text;
 
@@ -65,6 +66,7 @@ namespace Hochbaustatistik.Business
         public string Aktenstandort { get; set; }
         #endregion
 
+        [ExcludeFromCodeCoverage]
         public override string GenerateVorgang(Vorgang v)
         {          
             StringBuilder sbInpro = new StringBuilder();

Hochbaustatistik/Database/DatabaseOperations.cs

@@ -92,81 +92,27 @@ namespace Hochbaustatistik.Database
                     vorgangInproHochbau.ZdA = reader.GetValue(6).ToString();
 
                     vorgangInproHochbau.Eingangsdatum = reader.GetValue(7).ToString().Split(' ').FirstOrDefault();
-                    var tempDate = reader.GetValue(8).ToString();
                     
-                    if (tempDate == null)
+                    vorgangInproHochbau.Genehmigung = reader.GetValue(8).ToString() == null ? "" : reader.GetValue(8).ToString().Split(' ').FirstOrDefault();
-                    {
-                        vorgangInproHochbau.Genehmigung = "";
-                    }
-                    else
-                    {
-                        vorgangInproHochbau.Genehmigung = tempDate.Split(' ').FirstOrDefault();
-                    }
 
                     vorgangInproHochbau.Freigabe = reader.GetValue(9).ToString();
                     vorgangInproHochbau.Statistik_Relevant = reader.GetValue(10).ToString();
                     vorgangInproHochbau.Statistik1 = reader.GetValue(11).ToString();
                     vorgangInproHochbau.Statistik2 = reader.GetValue(12).ToString();
 
-                    tempDate = reader.GetValue(13).ToString();
+                    vorgangInproHochbau.Genehmigung_Gemeldet = reader.GetValue(13).ToString() == null ? "" : reader.GetValue(13).ToString().Split(' ').FirstOrDefault();
-                    if (tempDate == null)
-                    {
-                        vorgangInproHochbau.Genehmigung_Gemeldet = "";
-                    }
-                    else
-                    {
-                        vorgangInproHochbau.Genehmigung_Gemeldet = tempDate.Split(' ').FirstOrDefault();
-                    }
 
-                    tempDate = reader.GetValue(14).ToString();
+                    vorgangInproHochbau.Fertigstellung_Gemeldet = reader.GetValue(14).ToString() == null ? "" : reader.GetValue(14).ToString().Split(' ').FirstOrDefault();
-                    if (tempDate == null)
+                    vorgangInproHochbau.Baubeginn = reader.GetValue(15).ToString() == null ? "" : reader.GetValue(15).ToString().Split(' ').FirstOrDefault();
-                    {
-                        vorgangInproHochbau.Fertigstellung_Gemeldet = "";
-                    }
-                    else
-                    {
-                        vorgangInproHochbau.Fertigstellung_Gemeldet = tempDate.Split(' ').FirstOrDefault();
-                    }
 
-                    tempDate = reader.GetValue(15).ToString();
+                    vorgangInproHochbau.Teilrohbauabnahme = reader.GetValue(16).ToString() == null ? "" : reader.GetValue(16).ToString().Split(' ').FirstOrDefault();
-                    if (tempDate == null)
-                    {
-                        vorgangInproHochbau.Baubeginn = "";
-                    }
-                    else
-                    {
-                        vorgangInproHochbau.Baubeginn = tempDate.Split(' ').FirstOrDefault();
-                    }
-                    tempDate = reader.GetValue(16).ToString();
-                    if (tempDate == null)
-                    {
-                        vorgangInproHochbau.Teilrohbauabnahme = "";
-                    }
-                    else
-                    {
-                        vorgangInproHochbau.Teilrohbauabnahme = tempDate.Split(' ').FirstOrDefault();
-                    }
 
-                    tempDate = reader.GetValue(17).ToString();
+                    vorgangInproHochbau.Rohbauabnahme = reader.GetValue(17).ToString() == null ? "" : reader.GetValue(17).ToString().Split(' ').FirstOrDefault();
-                    if (tempDate == null)
-                    {
-                        vorgangInproHochbau.Rohbauabnahme = "";
-                    }
-                    else
-                    {
-                        vorgangInproHochbau.Rohbauabnahme = tempDate.Split(' ').FirstOrDefault();
-                    }
 
-                    tempDate = reader.GetValue(18).ToString();
+                    vorgangInproHochbau.Schlussabnahme = reader.GetValue(18).ToString() == null ? "" : reader.GetValue(18).ToString().Split(' ').FirstOrDefault();
-                    if (tempDate == null)
-                    {
-                        vorgangInproHochbau.Schlussabnahme = "";
-                    }
-                    else
-                    {
-                        vorgangInproHochbau.Schlussabnahme = tempDate.Split(' ').FirstOrDefault();
-                    }
 
+
+                    string tempDate = string.Empty;
                     if (tempDate == null)
                     {
                         vorgangInproHochbau.Vorjahr = "";

Hochbaustatistik/Utilities/DataOpsSingleton.cs

@@ -77,19 +77,6 @@ namespace Hochbaustatistik.Utilities
             return az;
         }
          
-        [ExcludeFromCodeCoverage]
-        public string ApplyLastAzToNewAz(string az, string currentName, string currentStrasse, LastData ld)
-        {
-            int linguName = String.Compare(currentName, ld.name, StringComparison.InvariantCultureIgnoreCase);
-            int linguStrasse = String.Compare(currentStrasse, ld.strasse, StringComparison.InvariantCultureIgnoreCase);
-
-            if (linguName > 0 || linguStrasse > 0)
-            {
-                az = ld.az;
-            }
-            return az;
-        }
-
         [ExcludeFromCodeCoverage]
         private void CheckForDoubleSlashAndCorrect(ref string az)
         {
@@ -186,7 +173,7 @@ namespace Hochbaustatistik.Utilities
         public bool IsValid(string az)
         {
             string pattern = @"\b(61[/][0-9]{1}[-][0-9]{1}[-][0-9]{6})";
-            Regex r = new Regex(pattern);
+            Regex r = new Regex(pattern,RegexOptions.None, TimeSpan.FromMilliseconds(5));
             bool matched = r.IsMatch(az);
             bool len = az.Length == 13;
             if (matched && len)
@@ -299,8 +286,7 @@ namespace Hochbaustatistik.Utilities
             VorgangInpro vorgangHochbauInpro = DatabaseOperations.ConnectAndExecuteHochbau(vorgang);
             if (vorgangHochbauInpro.IdStrasse == null)
             {
-                vorgangHochbauInpro = DatabaseOperations.ConnectAndExecuteHochbau(vorgang, Convert.ToInt32(identifikator));// Convert.ToInt32(ReshapeIdentifikator(hochbauListe.ElementAt(record).Identifikator)));
+                vorgangHochbauInpro = DatabaseOperations.ConnectAndExecuteHochbau(vorgang, Convert.ToInt32(identifikator));
-
             }
             if (vorgang.Aktenzeichen.Equals("NOT_CORRECTED"))
             {
@@ -316,7 +302,7 @@ namespace Hochbaustatistik.Utilities
         }
         #endregion
 
-        private class Nested
+        private static class Nested
         {
             // Tell C# compiler not to mark type as beforefieldinit (https://csharpindepth.com/articles/BeforeFieldInit)
             static Nested()
@@ -332,12 +318,4 @@ namespace Hochbaustatistik.Utilities
             _logger.Info("Singleton erzeugt.");
         }
     }       
-
-    public struct LastData
-    {
-        public string az;
-        public string ident;
-        public string name;
-        public string strasse;
-    }
 }

Hochbaustatistik/config/Jenkinsfile

@@ -4,7 +4,7 @@ pipeline
 	{
         node 
 		{
-            label 'Jenkins_Homer'
+            label 'jenkins_homer_agent'
         }                
 	}            
     stages 
@@ -14,16 +14,13 @@ pipeline
             environment
             {
                 dotnet ="C:\\Program Files\\dotnet\\"
-                scannerHome = "C:\\home\\jenkins\\tools\\hudson.plugins.sonar.MsBuildSQRunnerInstallation\\C_sonar-runner"                
+                
-                token = 'sqa_71202f188aacbe40492ba767cf3e841179d3f41a'
-                project = 'hochbaustatistik'
             }	
             steps 
 			{     
                   cleanWs()
                   checkout scm
                   bat "dotnet restore Hochbaustatistik\\Hochbaustatistik.sln"                  
-                  bat "${scannerHome}\\SonarScanner.MSBuild.exe begin /k:${project} /d:sonar.login=${token} /d:sonar.verbose=true /d:sonar.cs.opencover.reportsPaths=${WORKSPACE}\\coverage.xml"
                   bat "\"${tool 'MSBuild17'}\" ${WORKSPACE}\\Hochbaustatistik\\Hochbaustatistik.sln /t:Restore /t:Rebuild /p:Configuration=Debug"                       
              }
         }     
@@ -46,12 +43,14 @@ pipeline
             environment
             {    
                 scannerHome = "C:\\home\\jenkins\\tools\\hudson.plugins.sonar.MsBuildSQRunnerInstallation\\C_sonar-runner"
+                //token = '5f6fffb1bd335a10f8da70b77701bb73d6cd15cf'                
                 token = 'sqa_71202f188aacbe40492ba767cf3e841179d3f41a'
                 project = 'hochbaustatistik'                                            
             }	
             steps
             {
-                bat "${scannerHome}\\SonarScanner.MSBuild.exe end /d:sonar.login=${token}"
+                bat "${scannerHome}\\SonarScanner.MSBuild.exe begin /k:${project} /d:sonar.token=${token} /d:sonar.verbose=true /d:sonar.cs.opencover.reportsPaths=${WORKSPACE}\\coverage.xml
+                bat "${scannerHome}\\SonarScanner.MSBuild.exe end /d:sonar.token=${token}"
             }            
         }
         stage('Coverage')