FIx typos (#5185)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
This commit is contained in:
Родитель
8b766e3728
Коммит
6da4d9649d
|
@ -41,7 +41,7 @@ The proxy will create a file locally containing the data in the .assets folder i
|
||||||
|
|
||||||
When AZURE_TEST_MODE=PLAYBACK we invoke the playback/start endpoint on test-proxy which causes it to download the appropriate release indicated in assets.json if it is not there already, and replay the responses for the specific request based on the recorded data. When the test is done we call playback/stop
|
When AZURE_TEST_MODE=PLAYBACK we invoke the playback/start endpoint on test-proxy which causes it to download the appropriate release indicated in assets.json if it is not there already, and replay the responses for the specific request based on the recorded data. When the test is done we call playback/stop
|
||||||
|
|
||||||
## LIve Testing
|
## Live Testing
|
||||||
|
|
||||||
When AZURE_TEST_MODE=LIVE all requests are sent directly to their destination bypassing the proxy.
|
When AZURE_TEST_MODE=LIVE all requests are sent directly to their destination bypassing the proxy.
|
||||||
|
|
||||||
|
|
|
@ -5,7 +5,7 @@
|
||||||
#include <string>
|
#include <string>
|
||||||
#include <vector>
|
#include <vector>
|
||||||
/**
|
/**
|
||||||
* @brief THe Cryptography class provides a set of basic cryptographic primatives required
|
* @brief The Cryptography class provides a set of basic cryptographic primatives required
|
||||||
* by the attestation samples.
|
* by the attestation samples.
|
||||||
*/
|
*/
|
||||||
class Cryptography {
|
class Cryptography {
|
||||||
|
|
|
@ -9,7 +9,7 @@
|
||||||
|
|
||||||
namespace Azure { namespace Security { namespace Attestation { namespace _detail {
|
namespace Azure { namespace Security { namespace Attestation { namespace _detail {
|
||||||
/**
|
/**
|
||||||
* @brief THe Cryptography class provides a set of basic cryptographic functions required
|
* @brief The Cryptography class provides a set of basic cryptographic functions required
|
||||||
* by the attestation service client implementation and test collateral.
|
* by the attestation service client implementation and test collateral.
|
||||||
*
|
*
|
||||||
* It contains two subclasses: {@link Cryptography::AsymmetricKey}, which represents an Asymmetric
|
* It contains two subclasses: {@link Cryptography::AsymmetricKey}, which represents an Asymmetric
|
||||||
|
|
|
@ -276,7 +276,7 @@ namespace Azure { namespace Security { namespace Attestation { namespace _detail
|
||||||
auto exportedPublicKey = newCertificateKey->ExportPublicKey();
|
auto exportedPublicKey = newCertificateKey->ExportPublicKey();
|
||||||
auto publicKey = Cryptography::ImportPublicKey(exportedPublicKey);
|
auto publicKey = Cryptography::ImportPublicKey(exportedPublicKey);
|
||||||
// We know that Crypto::ImportPublicKey always returns an OpenSSLAsymmetricKey.
|
// We know that Crypto::ImportPublicKey always returns an OpenSSLAsymmetricKey.
|
||||||
// THis is a bit of a hack but it's an acceptable assumption to make.
|
// This is a bit of a hack but it's an acceptable assumption to make.
|
||||||
OpenSSLAsymmetricKey* key = static_cast<OpenSSLAsymmetricKey*>(publicKey.get());
|
OpenSSLAsymmetricKey* key = static_cast<OpenSSLAsymmetricKey*>(publicKey.get());
|
||||||
if (X509_set_pubkey(certificate.get(), key->GetKey().get()) != 1)
|
if (X509_set_pubkey(certificate.get(), key->GetKey().get()) != 1)
|
||||||
{
|
{
|
||||||
|
|
|
@ -304,7 +304,7 @@ namespace Azure { namespace Security { namespace Attestation { namespace Test {
|
||||||
}
|
}
|
||||||
|
|
||||||
// Verify that we get an exception if we try to set a policy management certificate on an AAD
|
// Verify that we get an exception if we try to set a policy management certificate on an AAD
|
||||||
// instance. THe primary purpose of this test is to increase code coverage numbers.
|
// instance. The primary purpose of this test is to increase code coverage numbers.
|
||||||
TEST_F(CertificateTests, VerifyFailedAddCertificate)
|
TEST_F(CertificateTests, VerifyFailedAddCertificate)
|
||||||
{
|
{
|
||||||
auto adminClient(CreateClient(ServiceInstanceType::AAD));
|
auto adminClient(CreateClient(ServiceInstanceType::AAD));
|
||||||
|
@ -331,7 +331,7 @@ namespace Azure { namespace Security { namespace Attestation { namespace Test {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
// Verify that we get an exception if we try to remove a policy management certificate on an AAD
|
// Verify that we get an exception if we try to remove a policy management certificate on an AAD
|
||||||
// instance. THe primary purpose of this test is to increase code coverage numbers.
|
// instance. The primary purpose of this test is to increase code coverage numbers.
|
||||||
TEST_F(CertificateTests, VerifyFailedRemoveCertificate)
|
TEST_F(CertificateTests, VerifyFailedRemoveCertificate)
|
||||||
{
|
{
|
||||||
auto adminClient(CreateClient(ServiceInstanceType::AAD));
|
auto adminClient(CreateClient(ServiceInstanceType::AAD));
|
||||||
|
|
|
@ -1259,7 +1259,7 @@ TEST_F(TestValueSerialization, SerializeList)
|
||||||
EXPECT_EQ(val.size(), 1);
|
EXPECT_EQ(val.size(), 1);
|
||||||
EXPECT_EQ(0x45, val[0]);
|
EXPECT_EQ(0x45, val[0]);
|
||||||
}
|
}
|
||||||
// THird form, serialized as first form.
|
// Third form, serialized as first form.
|
||||||
{
|
{
|
||||||
std::vector<uint8_t> testVector{0xd0, 0x00, 0x00, 0x00, 0x04, 0x00, 0x00, 0x00, 0x00};
|
std::vector<uint8_t> testVector{0xd0, 0x00, 0x00, 0x00, 0x04, 0x00, 0x00, 0x00, 0x00};
|
||||||
AmqpValue value{AmqpValue::Deserialize(testVector.data(), testVector.size())};
|
AmqpValue value{AmqpValue::Deserialize(testVector.data(), testVector.size())};
|
||||||
|
|
|
@ -36,7 +36,7 @@ namespace Azure { namespace Core {
|
||||||
/**
|
/**
|
||||||
* @brief Unique handle for WinHTTP HINTERNET handles.
|
* @brief Unique handle for WinHTTP HINTERNET handles.
|
||||||
*
|
*
|
||||||
* @note HINTERNET is declared as a "void *". THis means that this definition subsumes all other
|
* @note HINTERNET is declared as a "void *". This means that this definition subsumes all other
|
||||||
* `void *` types when used with Azure::Core::_internal::UniqueHandle.
|
* `void *` types when used with Azure::Core::_internal::UniqueHandle.
|
||||||
*
|
*
|
||||||
*/
|
*/
|
||||||
|
|
|
@ -1492,7 +1492,7 @@ namespace Azure { namespace Core {
|
||||||
namespace {
|
namespace {
|
||||||
// int g_ssl_crl_max_size_in_kb = 20;
|
// int g_ssl_crl_max_size_in_kb = 20;
|
||||||
/**
|
/**
|
||||||
* @brief THe Cryptography class provides a set of basic cryptographic primatives required
|
* @brief The Cryptography class provides a set of basic cryptographic primatives required
|
||||||
* by the attestation samples.
|
* by the attestation samples.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
|
|
|
@ -35,7 +35,7 @@ namespace Azure { namespace Core {
|
||||||
/**
|
/**
|
||||||
* @brief Unique handle for WinHTTP HINTERNET handles.
|
* @brief Unique handle for WinHTTP HINTERNET handles.
|
||||||
*
|
*
|
||||||
* @note HINTERNET is declared as a "void *". THis means that this definition subsumes all other
|
* @note HINTERNET is declared as a "void *". This means that this definition subsumes all other
|
||||||
* `void *` types when used with Azure::Core::_internal::UniqueHandle.
|
* `void *` types when used with Azure::Core::_internal::UniqueHandle.
|
||||||
*
|
*
|
||||||
*/
|
*/
|
||||||
|
|
|
@ -551,7 +551,7 @@ namespace Azure { namespace Core { namespace Test {
|
||||||
{
|
{
|
||||||
Azure::Core::Http::Policies::TransportOptions transportOptions;
|
Azure::Core::Http::Policies::TransportOptions transportOptions;
|
||||||
|
|
||||||
// FIrst verify connectivity to the test servers.
|
// First verify connectivity to the test servers.
|
||||||
transportOptions.EnableCertificateRevocationListCheck = false;
|
transportOptions.EnableCertificateRevocationListCheck = false;
|
||||||
HttpPipeline pipeline(CreateHttpPipeline(transportOptions));
|
HttpPipeline pipeline(CreateHttpPipeline(transportOptions));
|
||||||
|
|
||||||
|
|
|
@ -32,7 +32,7 @@ namespace Azure { namespace Messaging { namespace EventHubs { namespace Models {
|
||||||
* that contains it.*/
|
* that contains it.*/
|
||||||
std::string Name;
|
std::string Name;
|
||||||
|
|
||||||
/** THe identifier of the partition, unique to the Event Hub which contains it. */
|
/** The identifier of the partition, unique to the Event Hub which contains it. */
|
||||||
std::string PartitionId;
|
std::string PartitionId;
|
||||||
|
|
||||||
/** The first sequence number available for events in the partition.
|
/** The first sequence number available for events in the partition.
|
||||||
|
|
|
@ -241,7 +241,7 @@ namespace Azure { namespace Messaging { namespace EventHubs { namespace PerfTest
|
||||||
false},
|
false},
|
||||||
{"PaddingBytes",
|
{"PaddingBytes",
|
||||||
{"--paddingBytes"},
|
{"--paddingBytes"},
|
||||||
"THe number of bytes to send in each message body.",
|
"The number of bytes to send in each message body.",
|
||||||
1,
|
1,
|
||||||
false},
|
false},
|
||||||
{"partitionId",
|
{"partitionId",
|
||||||
|
|
Загрузка…
Ссылка в новой задаче