9cdd68e7c1
The patch was reverted 27d52cd86a because of failures in
TestWeakSymbols.py. These have now been addressed in D83552.
The original commit message was:
This function was documented to overwrite entries with D76111, which was
adding a couple of similar functions. However, this function (unlike the
functions added in that patch) was/is not actually overwriting variables
-- any pre-existing variables would get ignored.
This behavior does not seem to be intentional. In fact, before the refactor in
D41359, this function could introduce duplicate entries, which could
have very surprising effects both inside lldb and on other applications
(some applications would take the first value, some the second one; in
lldb, attempting to unset a variable could make the second variable
become active, etc.).
Overwriting seems to be the most reasonable behavior here, so change the
code to match documentation.
Differential Revision: https://reviews.llvm.org/D83306
136 lines
4.6 KiB
Python
136 lines
4.6 KiB
Python
"""Test the SBEnvironment APIs."""
|
|
|
|
|
|
|
|
from math import fabs
|
|
import lldb
|
|
from lldbsuite.test.decorators import *
|
|
from lldbsuite.test.lldbtest import *
|
|
from lldbsuite.test import lldbutil
|
|
|
|
class SBEnvironmentAPICase(TestBase):
|
|
|
|
mydir = TestBase.compute_mydir(__file__)
|
|
NO_DEBUG_INFO_TESTCASE = True
|
|
|
|
# We use this function to test both kind of accessors:
|
|
# . Get*AtIndex and GetEntries
|
|
def assertEqualEntries(self, env, entries):
|
|
self.assertEqual(env.GetNumValues(), len(entries))
|
|
for i in range(env.GetNumValues()):
|
|
name = env.GetNameAtIndex(i)
|
|
value = env.GetValueAtIndex(i)
|
|
self.assertIn(name + "=" + value, entries)
|
|
|
|
entries = env.GetEntries()
|
|
self.assertEqual(entries.GetSize(), len(entries))
|
|
for i in range(entries.GetSize()):
|
|
(name, value) = entries.GetStringAtIndex(i).split("=")
|
|
self.assertIn(name + "=" + value, entries)
|
|
|
|
|
|
|
|
@add_test_categories(['pyapi'])
|
|
def test_platform_environment(self):
|
|
env = self.dbg.GetSelectedPlatform().GetEnvironment()
|
|
# We assume at least PATH is set
|
|
self.assertNotEqual(env.Get("PATH"), None)
|
|
|
|
|
|
@add_test_categories(['pyapi'])
|
|
def test_launch_info(self):
|
|
target = self.dbg.CreateTarget("")
|
|
launch_info = target.GetLaunchInfo()
|
|
env = launch_info.GetEnvironment()
|
|
env_count = env.GetNumValues()
|
|
|
|
env.Set("FOO", "bar", overwrite=True)
|
|
self.assertEqual(env.GetNumValues(), env_count + 1)
|
|
|
|
# Make sure we only modify the copy of the launchInfo's environment
|
|
self.assertEqual(launch_info.GetEnvironment().GetNumValues(), env_count)
|
|
|
|
launch_info.SetEnvironment(env, append=True)
|
|
self.assertEqual(launch_info.GetEnvironment().GetNumValues(), env_count + 1)
|
|
|
|
env.Set("FOO", "baz", overwrite=True)
|
|
launch_info.SetEnvironment(env, append=True)
|
|
self.assertEqual(launch_info.GetEnvironment().GetNumValues(), env_count + 1)
|
|
self.assertEqual(launch_info.GetEnvironment().Get("FOO"), "baz")
|
|
|
|
# Make sure we can replace the launchInfo's environment
|
|
env.Clear()
|
|
env.Set("BAR", "foo", overwrite=True)
|
|
env.PutEntry("X=y")
|
|
launch_info.SetEnvironment(env, append=False)
|
|
self.assertEqualEntries(launch_info.GetEnvironment(), ["BAR=foo", "X=y"])
|
|
|
|
|
|
@add_test_categories(['pyapi'])
|
|
def test_target_environment(self):
|
|
env = self.dbg.GetSelectedTarget().GetEnvironment()
|
|
# There is no target, so env should be empty
|
|
self.assertEqual(env.GetNumValues(), 0)
|
|
self.assertEqual(env.Get("PATH"), None)
|
|
|
|
target = self.dbg.CreateTarget("")
|
|
env = target.GetEnvironment()
|
|
path = env.Get("PATH")
|
|
# Now there's a target, so at least PATH should exist
|
|
self.assertNotEqual(path, None)
|
|
|
|
# Make sure we are getting a copy by modifying the env we just got
|
|
env.PutEntry("PATH=#" + path)
|
|
self.assertEqual(target.GetEnvironment().Get("PATH"), path)
|
|
|
|
@add_test_categories(['pyapi'])
|
|
def test_creating_and_modifying_environment(self):
|
|
env = lldb.SBEnvironment()
|
|
|
|
self.assertEqual(env.Get("FOO"), None)
|
|
self.assertEqual(env.Get("BAR"), None)
|
|
|
|
# We also test empty values
|
|
self.assertTrue(env.Set("FOO", "", overwrite=False))
|
|
env.Set("BAR", "foo", overwrite=False)
|
|
|
|
self.assertEqual(env.Get("FOO"), "")
|
|
self.assertEqual(env.Get("BAR"), "foo")
|
|
|
|
self.assertEqual(env.GetNumValues(), 2)
|
|
|
|
self.assertEqualEntries(env, ["FOO=", "BAR=foo"])
|
|
|
|
# Make sure modifications work
|
|
self.assertFalse(env.Set("FOO", "bar", overwrite=False))
|
|
self.assertEqual(env.Get("FOO"), "")
|
|
|
|
env.PutEntry("FOO=bar")
|
|
self.assertEqual(env.Get("FOO"), "bar")
|
|
|
|
self.assertEqualEntries(env, ["FOO=bar", "BAR=foo"])
|
|
|
|
# Make sure we can unset
|
|
self.assertTrue(env.Unset("FOO"))
|
|
self.assertFalse(env.Unset("FOO"))
|
|
self.assertEqual(env.Get("FOO"), None)
|
|
|
|
# Test SetEntries
|
|
entries = lldb.SBStringList()
|
|
entries.AppendList(["X=x", "Y=y"], 2)
|
|
|
|
env.SetEntries(entries, append=True)
|
|
self.assertEqualEntries(env, ["BAR=foo", "X=x", "Y=y"])
|
|
|
|
env.SetEntries(entries, append=False)
|
|
self.assertEqualEntries(env, ["X=x", "Y=y"])
|
|
|
|
entries.Clear()
|
|
entries.AppendList(["X=y", "Y=x"], 2)
|
|
env.SetEntries(entries, append=True)
|
|
self.assertEqualEntries(env, ["X=y", "Y=x"])
|
|
|
|
# Test clear
|
|
env.Clear()
|
|
self.assertEqualEntries(env, [])
|