php - 正确的单元测试

coder 2024-04-06 原文

我开始在这个项目中使用单元和功能测试，因此我有一些问题:

我正在使用 symfony php 框架。我有一个像 LDAP ORM 服务这样的原则。

此外，我有一个用户存储库(作为一项服务)，它依赖于 LDAP ORM 服务、记录器和验证服务。

现在我想为 UserRepo 的 addUser 函数编写一个单元测试。它将在内部调用:getNewUidNumber、userToEntities、doesUserExist 和 getUserByUid。

我的问题是: 我应该模拟所有这些内部函数来测试 addUser 函数吗？这会违背单元测试的想法吗(只测试 API)。

或者我应该只模拟 LDAP ORM 服务、Logger 和验证服务，以便该类调用所有内部函数？但这会导致带有大量模拟的巨大测试函数，因为我必须模拟所有内部存储库调用的存储库。

或者我应该启动 symfony 内核并使用 ServiceContainer 将 ORM LDAP 服务与真实的测试数据库一起使用。但这不是功能测试而不是单元测试吗？我听说在一个测试中有这么多的依赖是不好的。所以我认为使用整个 serviceContainer 会很糟糕。

添加用户:

public function addUser(User $user)
{
    $pbnlAccount = $this->userToEntities($user);

    if(!$this->doesUserExist($user)) {
        $pbnlAccount->setUidNumber($this->getNewUidNumber());
        $this->ldapEntityManager->persist($pbnlAccount);
        $this->ldapEntityManager->flush();
    }
    else {
        throw new UserAlreadyExistException("The user ".$user->getUid()." already exists.");
    }

    return $this->getUserByUid($user->getUid());
}

更多代码，比如内部函数: https://gist.github.com/NKPmedia/4a6ee55b6bb96e8af409debd98950678

谢谢保罗

最佳答案

首先，如果可以的话，我想稍微重写一下方法。

public function addUser(User $user)
{
    if ($this->doesUserExist($user)) {
        throw new UserAlreadyExistException("The user ".$user->getUid()." already exists.");
    }

    // ... shortened for brevity
    $pbnlAccount = $this->userToEntities($user);
    $this->ldapEntityManager->persist($pbnlAccount);
}

另一个相关的方法是:

private function doesUserExist(User $user)
{
    $users = $this->ldapRepository->findByUid($user->getUid());
    return count($users) === 1;
}

立即我们可以看到我们基本上有两个测试:

我们测试该方法在用户存在时抛出
我们测试如果用户不存在，该方法会保留一个 PbnlAccount。

如果您不明白我们为什么要进行这两个测试，请注意此方法中有两种可能的“流”:一种是执行 if 语句内的 block ，另一种是不执行它。

让我们解决第一个问题:

public function testAddUserThrowsWhenUserExistsAlready()
{
    $user = new User();
    $user->setUid('123');

    $ldapRepositoryMock = $this->createMock(LdapRepository::class);
    $ldapRepositoryMock
        ->method('findByUid')
        ->expects($this->once())
        ->with('123')
        ->willReturn(new PbnlAccount());

    $userRepository = new UserRepository($ldapRepositoryMock);

    $this->expectException(UserAlreadyExistException::class);
    $userRepository->addUser($user);    
}

第二个测试留给读者作为练习:)

是的你将必须在你的情况下做一些 mock 。在这种情况下，您需要同时模拟 LdapRepository 和 LdapEntityManager。

注意 1:此代码可能无法运行，因为我不知道您的代码库的确切细节(而且我是凭空写下的)，但这不是重点。关键是您要测试异常。

注2: 我会将您的函数重命名为 createNewPbnlAccountForUser(User $user)，它更长，但更能描述它的实际作用。

注3: 我不确定你为什么要返回 $this->getUserByUid() 因为这似乎是多余的(你已经有 User 了)，所以我忽略了这种情况。

关于php - 正确的单元测试，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/46164760/

php 正确 user gt this symfony unit-testing ldap phpunit

有关php - 正确的单元测试的更多相关文章

ruby-on-rails - 使用 Ruby on Rails 进行自动化测试 - 最佳实践 - 2
很好奇，就使用rubyonrails自动化单元测试而言，你们正在做什么？您是否创建了一个脚本来在cron中运行rake作业并将结果邮寄给您？git中的预提交Hook？只是手动调用？我完全理解测试，但想知道在错误发生之前捕获错误的最佳实践是什么。让我们理所当然地认为测试本身是完美无缺的，并且可以正常工作。下一步是什么以确保他们在正确的时间将可能有害的结果传达给您？最佳答案不确定您到底想听什么，但是有几个级别的自动代码库控制:在处理某项功能时，您可以使用类似autotest的内容获得关于哪些有效，哪些无效的即时反馈。要确保您的提
ruby - 使用 C 扩展开发 rubygem 时，如何使用 Rspec 在本地进行测试？ - 2
我正在编写一个包含C扩展的gem。通常当我写一个gem时，我会遵循TDD的过程，我会写一个失败的规范，然后处理代码直到它通过，等等......在“ext/mygem/mygem.c”中我的C扩展和在gemspec的“扩展”中配置的有效extconf.rb，如何运行我的规范并仍然加载我的C扩展？当我更改C代码时，我需要采取哪些步骤来重新编译代码？这可能是个愚蠢的问题，但是从我的gem的开发源代码树中输入“bundleinstall”不会构建任何native扩展。当我手动运行rubyext/mygem/extconf.rb时，我确实得到了一个Makefile(在整个项目的根目录中)，然后当
ruby - Ruby 的 Hash 在比较键时使用哪种相等性测试？ - 2
我有一个围绕一些对象的包装类，我想将这些对象用作散列中的键。包装对象和解包装对象应映射到相同的键。一个简单的例子是这样的:classAattr_reader:xdefinitialize(inner)@inner=innerenddefx;@inner.x;enddef==(other)@inner.x==other.xendenda=A.new(o)#oisjustanyobjectthatallowso.xb=A.new(o)h={a=>5}ph[a]#5ph[b]#nil,shouldbe5ph[o]#nil,shouldbe5我试过==、===、eq?并散列所有无济于事。
ruby-on-rails - 如何使用 instance_variable_set 正确设置实例变量？ - 2
我正在查看instance_variable_set的文档并看到给出的示例代码是这样做的:obj.instance_variable_set(:@instnc_var,"valuefortheinstancevariable")然后允许您在类的任何实例方法中以@instnc_var的形式访问该变量。我想知道为什么在@instnc_var之前需要一个冒号:。冒号有什么作用？最佳答案我的第一直觉是告诉你不要使用instance_variable_set除非你真的知道你用它做什么。它本质上是一种元编程工具或绕过实例变量可见性的黑客攻击
ruby - RSpec - 使用测试替身作为 block 参数 - 2
我有一些Ruby代码，如下所示:Something.createdo|x|x.foo=barend我想编写一个测试，它使用double代替block参数x，这样我就可以调用:x_double.should_receive(:foo).with("whatever").这可能吗？最佳答案 specify'something'dox=doublex.should_receive(:foo=).with("whatever")Something.should_receive(:create).and_yield(x)#callthere
ruby - Sinatra:运行 rspec 测试时记录噪音 - 2
Sinatra新手；我正在运行一些rspec测试，但在日志中收到了一堆不需要的噪音。如何消除日志中过多的噪音？我仔细检查了环境是否设置为:test，这意味着记录器级别应设置为WARN而不是DEBUG。spec_helper:require"./app"require"sinatra"require"rspec"require"rack/test"require"database_cleaner"require"factory_girl"set:environment,:testFactoryGirl.definition_file_paths=%w{./factories./test/
ruby-on-rails - 迷你测试错误 : "NameError: uninitialized constant" - 2
我遵循MichaelHartl的“RubyonRails教程:学习Web开发”，并创建了检查用户名和电子邮件长度有效性的测试(名称最多50个字符，电子邮件最多255个字符)。test/helpers/application_helper_test.rb的内容是:require'test_helper'classApplicationHelperTest在运行bundleexecraketest时，所有测试都通过了，但我看到以下消息在最后被标记为错误:ERROR["test_full_title_helper",ApplicationHelperTest,1.820016791]test
ruby - 即使失败也继续进行多主机测试 - 2
我已经构建了一些serverspec代码来在多个主机上运行一组测试。问题是当任何测试失败时，测试会在当前主机停止。即使测试失败，我也希望它继续在所有主机上运行。Rakefile:namespace:specdotask:all=>hosts.map{|h|'spec:'+h.split('.')[0]}hosts.eachdo|host|begindesc"Runserverspecto#{host}"RSpec::Core::RakeTask.new(host)do|t|ENV['TARGET_HOST']=hostt.pattern="spec/cfengine3/*_spec.r
ruby-on-rails - 如何使辅助方法在 Rails 集成测试中可用？ - 2
我在app/helpers/sessions_helper.rb中有一个帮助程序文件，其中包含一个方法my_preference，它返回当前登录用户的首选项。我想在集成测试中访问该方法。例如，这样我就可以在测试中使用getuser_path(my_preference)。在其他帖子中，我读到这可以通过在测试文件中包含requiresessions_helper来实现，但我仍然收到错误NameError:undefinedlocalvariableormethod'my_preference'.我做错了什么？require'test_helper'require'sessions_hel
ruby-on-rails - 正确的 Rails 2.1 做事方式 - 2
question的一些答案关于redirect_to让我想到了其他一些问题。基本上，我正在使用Rails2.1编写博客应用程序。我一直在尝试自己完成大部分工作(因为我对Rails有所了解)，但在需要时会引用Internet上的教程和引用资料。我设法让一个简单的博客正常运行，然后我尝试添加评论。靠我自己，我设法让它进入了可以从script/console添加评论的阶段，但我无法让表单正常工作。我遵循的其中一个教程建议在帖子Controller中创建一个“评论”操作，以添加评论。我的问题是:这是“标准”方式吗？我的另一个问题的答案之一似乎暗示应该有一个CommentsController参

php - 正确的单元测试

有关php - 正确的单元测试的更多相关文章

随机推荐