Enhancement CiscoDevNet#853

ygorelik · Jan 30, 2019 · bebd554 · bebd554
1 parent c9e4a2e
commit bebd554
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 4 deletions.
diff --git a/.gitignore b/.gitignore
@@ -39,4 +39,3 @@ protobuf-3.5.0
 sdk/python/gnmi/build
 sdk/python/gnmi/dist
 sdk/python/gnmi/ydk_service_gnmi.egg-info
-sdk/cpp/core/tests/repository/*
diff --git a/.travis.yml b/.travis.yml
@@ -8,7 +8,7 @@ matrix:
      dist: trusty
      group: edge
    - os: osx
-     osx_image: xcode8.3
+     osx_image: xcode7.3
      language: generic
 #   - os: osx
 #     env: GNMI=true

diff --git a/sdk/cpp/core/src/path/repository.cpp b/sdk/cpp/core/src/path/repository.cpp
@@ -235,7 +235,12 @@ ly_ctx* ydk::path::RepositoryPtr::create_ly_context()
         {
             for(auto model_provider : get_model_providers())
             {
-                path+="/"+model_provider->get_hostname_port();
+                //path+="/"+model_provider->get_hostname_port();
+                auto hostname_port = model_provider->get_hostname_port();
+                auto underscore_pos = hostname_port.rfind("_");
+                if (underscore_pos != std::string::npos)
+                    hostname_port = hostname_port.substr(0, underscore_pos);
+                path += "/" + hostname_port;
                 break;
             }
         }

diff --git a/sdk/cpp/gnmi/tests/test_gnmi_provider.cpp b/sdk/cpp/gnmi/tests/test_gnmi_provider.cpp
@@ -15,6 +15,10 @@
  ------------------------------------------------------------------*/
 #include <string.h>
 #include <iostream>
+#include <pwd.h>
+#include <unistd.h>
+#include <sys/types.h>
+#include <sys/stat.h>
 
 #include <ydk/gnmi_provider.hpp>
 #include <ydk/gnmi_service.hpp>
@@ -45,9 +49,30 @@ TEST_CASE("GNMICreateNoRepo")
 	CHECK_NOTHROW(provider.get_encoding());
 }
 
+static std::string get_temp_model_path()
+{
+    const char *homeDir = getenv("HOME");
+    if (!homeDir) {
+        struct passwd* pwd = getpwuid(getuid());
+        if (pwd)
+           homeDir = pwd->pw_dir;
+    }
+
+    std::ostringstream models_path{};
+    models_path << homeDir << "/.ydk/temp";
+    auto path = models_path.str();
+
+    struct stat st;
+    memset(&st, 0, sizeof(struct stat));
+    if (stat(path.c_str(), &st) == -1) {
+        mkdir(path.c_str(), 0755);
+    }
+    return path;
+}
+
 TEST_CASE("GNMICreateWithNoFiles")
 {
-	ydk::path::Repository repo{};
+	ydk::path::Repository repo{get_temp_model_path()};
 	gNMIServiceProvider provider{repo, "127.0.0.1", 50051, "admin", "admin"};
 	gNMIService gs{};