Vulnerability in EPiServer.Forms

Try our conversational search powered by Generative AI!

Stefan Forsberg
Nov 10, 2010
(2 votes)

Making EPiServer code testable – the mild mannered rant

As I mentioned in my last post writing tests for your code isn’t hard. The big “but” you know is coming is that this is valid only if the framework you’re writing code for/in is built to be testable. Just as an example, to be able to write the tests I wanted for the scenario outlined in the previous post I had to write this code.

   1: public interface ILanguageManagerFacade
   2:     {
   3:         string TranslateFallback(string key, string fallback);
   4:         string Translate(string key, string language);
   5:     }
   1: public class LanguageManagerFacade : ILanguageManagerFacade
   2:     {
   3:         public string TranslateFallback(string key, string fallback)
   4:         {
   5:             return LanguageManager.Instance.TranslateFallback(key, fallback);
   6:         }
   8:         public string Translate(string key, string language)
   9:         {
  10:             return LanguageManager.Instance.Translate(key, language);
  11:         }
  12:     }
   1: public interface IPageLanguageSettingFacade
   2:     {
   3:         PageLanguageSetting Load(PageReference pageReference, string language);
   4:     }
   1: public class PageLanguageSettingFacade : IPageLanguageSettingFacade
   2: {
   3:     public PageLanguageSetting Load(PageReference pageReference, string language)
   4:     {
   5:         return PageLanguageSetting.Load(pageReference, language);
   6:     }
   7: }
   1: public interface IPageDataFromHttpHandlerParser
   2:     {
   3:         PageData GetPage();
   4:     }
   1: public class PageDataFromHttpHandlerParser : IPageDataFromHttpHandlerParser
   2:     {
   3:         private readonly HttpContextBase _httpContext;
   5:         public PageDataFromHttpHandlerParser(HttpContextBase httpContext)
   6:         {
   7:             _httpContext = httpContext;
   8:         }
  10:         public PageData GetPage()
  11:         {
  12:             var page = _httpContext.Handler as PageBase;
  14:             return page.CurrentPage;
  15:         }
  16:     }
   1: public class TranslateWithFallbackLanguageBranch
   2:     {
   3:         private readonly ILanguageManagerFacade _languageManagerFacade;
   4:         private readonly IGetFallbackLanguage _getFallbackLanguage;
   6:         public TranslateWithFallbackLanguageBranch(ILanguageManagerFacade languageManagerFacade, IGetFallbackLanguage getFallbackLanguage)
   7:         {
   8:             _languageManagerFacade = languageManagerFacade;
   9:             _getFallbackLanguage = getFallbackLanguage;
  10:         }
  12:         public string Translate(string key)
  13:         {
  14:             string fallbackLanguageId = _getFallbackLanguage.GetFallbackLanguage();
  16:             return _languageManagerFacade.TranslateFallback(key, _languageManagerFacade.Translate(key, fallbackLanguageId));
  17:         }
  18:     }
   1: public interface IGetFallbackLanguage
   2:     {
   3:         string GetFallbackLanguage();
   4:     }
   1: public class GetFallbackLanguageFromRootBasedOnCurrentPage : IGetFallbackLanguage
   2:     {
   3:         private readonly IPageLanguageSettingFacade _pageLanguageSettingFacade;
   4:         private readonly IPageReferenceFacade _pageReferenceFacade;
   5:         private readonly IPageDataFromHttpHandlerParser _pageDataFromHttpHandlerParser;
   7:         public GetFallbackLanguageFromRootBasedOnCurrentPage(IPageLanguageSettingFacade pageLanguageSettingFacade, IPageReferenceFacade pageReferenceFacade, IPageDataFromHttpHandlerParser pageDataFromHttpHandlerParser)
   8:         {
   9:             _pageLanguageSettingFacade = pageLanguageSettingFacade;
  10:             _pageReferenceFacade = pageReferenceFacade;
  11:             _pageDataFromHttpHandlerParser = pageDataFromHttpHandlerParser;
  12:         }
  14:         public string GetFallbackLanguage()
  15:         {
  16:             // Get PageBase using the current handler
  17:             var page = _pageDataFromHttpHandlerParser.GetPage();
  19:             // Get the page language setting from the root page, using the current page's language
  20:             // This is where the replacement and fallback languages are defined
  21:             PageLanguageSetting pageLanguageSetting = _pageLanguageSettingFacade.Load(_pageReferenceFacade.RootPage, page.LanguageID);
  23:             if (pageLanguageSetting != null)
  24:             {
  25:                 // Check if there actually is a fallback language defined
  26:                 // If so, use the first one
  27:                 if (pageLanguageSetting.LanguageBranchFallback.Length > 0)
  28:                     return pageLanguageSetting.LanguageBranchFallback[0];
  29:             }
  31:             // If there is no fallback language defined then we return the current page's language ID
  32:             return page.LanguageID;
  33:         }
  34:     }

I also made use of some EPiAbstraction classes.

The code above is not too interesting but will be discussed in a later post. The point I’m trying to make is that you often have to spend a lot more time on making _episerver_ code testable than writing the tests themselves. This sort of makes the case of the “testing is easy” argument a hard sell.


Some closing thoughts

In all fairness (and please correct me if I’m way off base here) I don’t think the EPiServer team in the past has had too many requests from developers to focus on this area. I know that I personally didn’t care too much about it 3 years ago before I was introduced to the involved concepts. But I feel that as the sites we develop get more and more complex being able to, for instance, develop using TDD is becoming a must have.

So, here’s hoping for a more testable future.

Nov 10, 2010


Kjetil Simensen
Kjetil Simensen Nov 11, 2010 07:52 AM


In our current project we're introducing unit testing and we're finding it very useful. but we have some difficulties testing UnifiedFiles and HostingEnvironment related code. Have you had any experience testing this?


Stefan Forsberg
Stefan Forsberg Nov 11, 2010 08:15 AM

Hello Kjetil,
There's a ready-to-use abstraction/wrapper for UnifiedFile available in epiabstraction ( so that should save you from having to create it yourself.

Other than that I find it's easier to discuss this when I know more about what you're trying to do, what business requirements you have etc. Feel free to contact me at stefan dot forsberg at cloudnine dot se.

Kjetil Simensen
Kjetil Simensen Nov 11, 2010 08:35 AM

Ah, we must have missed that part of the epiabstraction! We'll have a close look now that we know it's should be in there somewhere :)

Thanks, will look into it and see if it solves our problems.


Nov 11, 2010 11:04 AM

Nice. Looking forward to the next post!

Please login to comment.
Latest blogs
Google Read Aloud Reload Problems

Inclusive web experiences greatly benefit from accessibility features such as Google Read Aloud. This tool, which converts text into speech, enable...

Luc Gosso (MVP) | Dec 4, 2023 | Syndicated blog

Import Blobs and Databases to Integration Environments

In this blog, we are going to explore some new extensions to the Deployment API in DXP Cloud Services, specifically the ability to import databases...

Elias Lundmark | Dec 4, 2023

Join the Work Smarter Webinar: Working with the Power of Configured Commerce (B2B) Customer Segmentation December 7th

Join this webinar and learn about customer segmentation – how to best utilize it, how to use personalization to differentiate segmentation and how...

Karen McDougall | Dec 1, 2023

Getting Started with Optimizely SaaS Core and Next.js Integration: Creating Content Pages

The blog post discusses the creation of additional page types with Next.js and Optimizely SaaS Core. It provides a step-by-step guide on how to...

Francisco Quintanilla | Dec 1, 2023 | Syndicated blog

Stop Managing Humans in Your CMS

Too many times, a content management system becomes a people management system. Meaning, an organization uses the CMS to manage all the information...

Deane Barker | Nov 30, 2023

A day in the life of an Optimizely Developer - Optimizely CMS 12: The advantages and considerations when exploring an upgrade

GRAHAM CARR - LEAD .NET DEVELOPER, 28 Nov 2023 In 2022, Optimizely released CMS 12 as part of its ongoing evolution of the platform to help provide...

Graham Carr | Nov 28, 2023